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

Make it possible to download a ZIP file containing QR codes of bulk selected netboxes/rooms #2899

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

johannaengland
Copy link
Contributor

@johannaengland johannaengland commented May 6, 2024

The last step to #2586.

It does not currently offer the ZIP file to download. I was able to create the ZIP file and am able to access it in the frontend (which I've shown by setting the ZIP file as bytes as the source of the added anchor element), but I do not know how to make it possible for the user to download it from there.

Screenshots - before and after

Before:

Screenshot from 2024-05-16 13-04-24
Screenshot from 2024-05-16 13-04-41

After:

Screenshot from 2024-05-16 13-08-03
Screenshot from 2024-05-16 13-08-15

@johannaengland johannaengland self-assigned this May 6, 2024
@johannaengland johannaengland changed the title Feature/bulk download qr codes Make it possible to download a ZIP file containing QR codes of bulk selected netboxes/rooms May 6, 2024
Copy link

github-actions bot commented May 6, 2024

Test results

     12 files       12 suites   11m 44s ⏱️
3 324 tests 3 324 ✔️ 0 💤 0
9 447 runs  9 447 ✔️ 0 💤 0

Results for commit d21e516.

♻️ This comment has been updated with latest results.

@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch from afdc234 to 1bfc71e Compare May 7, 2024 07:15
Copy link

codecov bot commented May 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.60%. Comparing base (0a90566) to head (fb3e170).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2899      +/-   ##
==========================================
+ Coverage   56.35%   56.60%   +0.24%     
==========================================
  Files         603      603              
  Lines       43862    43916      +54     
  Branches       48       48              
==========================================
+ Hits        24719    24859     +140     
+ Misses      19131    19045      -86     
  Partials       12       12              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch 2 times, most recently from d21e516 to 17bc18f Compare May 7, 2024 09:55
@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch from 17bc18f to 9d4bb24 Compare May 16, 2024 07:53
@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch from 9d4bb24 to f75b9eb Compare May 16, 2024 10:45
@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch from f75b9eb to 2be6e05 Compare July 3, 2024 12:48
@CLAassistant
Copy link

CLAassistant commented Jul 3, 2024

CLA assistant check
All committers have signed the CLA.

@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch from 2be6e05 to fb3e170 Compare August 7, 2024 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants