Skip to content
Snippets Groups Projects

Resolve "Refactor file up- and download"

What does this MR do (summary)?

Closes #51 (closed), #56 (closed)

  • Paths are no longer assumed to be relative
  • All paths can now be configured through the application.yaml
  • Single files are no longer zipped
  • Downloading multiple submissions is no longer a zip of zips, but rather just directories in a zip
  • Downloading multiple submissions now gives a zip in the following format:
submissions.zip
    assignment_1/
        group_1/
            submission_1/
                file1.s
                file2.s
        group_2/
            submission_2/
                file.txt
    assignment_2/
        ...

Does this MR meet the acceptance criteria?

  • The main application was run to make sure the server still runs without errors.
  • Tests were written to verify the behaviour of this code.
  • Javadoc and other comments were added to make the code understandable.
  • The code adheres to the style guidelines.
Edited by Ruben Backx

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Otto Visser
  • Ruben Backx added 1 commit

    added 1 commit

    • 712a0d3b - Make file storage use only two directories

    Compare with previous version

  • Ruben Backx added 4 commits

    added 4 commits

    Compare with previous version

  • Ruben Backx added 3 commits

    added 3 commits

    • 5f87aa15 - Fix CI
    • a66936de - Redo submission uploading and downloading
    • f6139abc - Make file storage use only two directories

    Compare with previous version

  • Ruben Backx added 8 commits

    added 8 commits

    Compare with previous version

  • Otto Visser resolved all threads

    resolved all threads

  • Otto Visser approved this merge request

    approved this merge request

  • Otto Visser started a merge train

    started a merge train

  • Otto Visser mentioned in commit c94d3f69

    mentioned in commit c94d3f69

  • merged

  • Please register or sign in to reply
    Loading