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 60.81%. Comparing base (237e1b6) to head (3ad0cc9).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2899      +/-   ##
==========================================
+ Coverage   60.76%   60.81%   +0.05%     
==========================================
  Files         606      606              
  Lines       43794    43847      +53     
  Branches       48       48              
==========================================
+ Hits        26610    26667      +57     
+ Misses      17172    17168       -4     
  Partials       12       12              

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

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@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 fb3e170 to 8567928 Compare March 7, 2025 14:12
@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch from 8567928 to 3ad0cc9 Compare March 7, 2025 14:16
Copy link

sonarqubecloud bot commented Mar 7, 2025

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.

2 participants