Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hide binary files on the user interface #332

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hannesbraun
Copy link
Contributor

When binary files are submitted, they are currently also shown on the user interface with the content "Binary data". These binary files don't really need to be shown on the user interface. This makes finding the file you're searching for easier. If you need to access to binary files, you can still download the (whole) solution.
The only drawbacks with this approach are that you'll lose the ability to view what binary files were submitted. Also you can't just download a single binary file through the user interface anymore.

I'd prefer to have binary files hidden on the user interface instead of having the current situation. But let me know what you think about this. Maybe my approach would break something essential for someone else.

@ifrh
Copy link
Contributor

ifrh commented Jul 29, 2022

How related are the changes to the discussion in feature request "List of submission files in the script checker" #181 ?

@hannesbraun
Copy link
Contributor Author

How related are the changes to the discussion in feature request "List of submission files in the script checker" #181 ?

This should not really be related to that discussion as my changes only affect what's being shown on the user interface. The discussion was about passing the list of submitted files to the script checker.

@hannesbraun
Copy link
Contributor Author

I noticed that the Praktomat is also able to display images and PDFs. With a2985e4, these are also displayed again on the web interface.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants