Skip to content

Add post request to explicitely prepare download archive#499

Merged
varmar05 merged 4 commits intodevelopfrom
post_prepare_download
Aug 27, 2025
Merged

Add post request to explicitely prepare download archive#499
varmar05 merged 4 commits intodevelopfrom
post_prepare_download

Conversation

@harminius
Copy link
Contributor

@harminius harminius commented Aug 18, 2025

Follows up #497

Hitting the download button:

  1. POST - triggers the celery job to create the project zip
  2. HEAD - polls server until the archive is ready (200) and
  3. hidden GET downloads the archive

➕ Improvements:
Only one celery job can be created from a browser

@harminius harminius requested a review from MarcelGeo August 18, 2025 16:09
@coveralls
Copy link

coveralls commented Aug 19, 2025

Pull Request Test Coverage Report for Build 17243023533

Details

  • 36 of 36 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 94.316%

Totals Coverage Status
Change from base Build 17033262050: 0.0%
Covered Lines: 7533
Relevant Lines: 7987

💛 - Coveralls

@harminius harminius requested a review from varmar05 August 19, 2025 11:47
@varmar05 varmar05 merged commit 076202d into develop Aug 27, 2025
4 checks passed
@RastoHu
Copy link

RastoHu commented Aug 29, 2025

test: download project from web

@RastoHu
Copy link

RastoHu commented Aug 29, 2025

@harminius
invalid error message for large projects (more than 10GB):

expected error message should be:
image

@MarcelGeo
Copy link
Collaborator

@harminius please fix that by adding handling of error also for that POST request.

@MarcelGeo
Copy link
Collaborator

@RastoHu could you double check this on staging server? thanks.

@MarcelGeo MarcelGeo deleted the post_prepare_download branch September 24, 2025 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants