Skip to content

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

Merged
johannaengland merged 3 commits intoUninett:masterfrom
johannaengland:feature/bulk-download-qr-codes
Aug 12, 2025
Merged

Make it possible to download a ZIP file containing QR codes of bulk selected netboxes/rooms#2899
johannaengland merged 3 commits intoUninett:masterfrom
johannaengland:feature/bulk-download-qr-codes

Conversation

@johannaengland
Copy link
Copy Markdown
Contributor

@johannaengland johannaengland commented May 6, 2024

The last step to #2586. Dependent on #2897.

Screenshots - before and after

Before:

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

After:

image

@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
@github-actions
Copy link
Copy Markdown

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
@codecov
Copy link
Copy Markdown

codecov bot commented May 7, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.89%. Comparing base (4961212) to head (8a5d8a4).
⚠️ Report is 523 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2899      +/-   ##
==========================================
+ Coverage   60.87%   60.89%   +0.01%     
==========================================
  Files         607      607              
  Lines       43953    43994      +41     
  Branches       48       48              
==========================================
+ Hits        26758    26790      +32     
- Misses      17183    17192       +9     
  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
Copy Markdown

CLAassistant commented Jul 3, 2024

CLA assistant check
All committers have signed the CLA.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Mar 7, 2025

@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch 2 times, most recently from 12dce3f to eb6af1e Compare July 25, 2025 10:06
@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch from eb6af1e to dc570c8 Compare July 28, 2025 13:12
@johannaengland johannaengland requested review from a team and removed request for hmpf July 28, 2025 13:14
@johannaengland johannaengland removed the request for review from stveit July 28, 2025 13:14
@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch 2 times, most recently from abfebcd to 81d9f75 Compare July 28, 2025 13:28
@johannaengland johannaengland requested a review from jorund1 July 30, 2025 10:01
@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch from 352ad68 to 35b2469 Compare July 30, 2025 10:04
@johannaengland johannaengland marked this pull request as ready for review July 30, 2025 10:06
@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch from 35b2469 to 98b6d47 Compare August 4, 2025 09:22
@johannaengland
Copy link
Copy Markdown
Contributor Author

Needed to rebase on #2897 since it changes some function signatures that are also used in here

@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch 2 times, most recently from 50eccc0 to 42247b0 Compare August 5, 2025 08:48
@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch from 42247b0 to 14681ed Compare August 11, 2025 12:44
Copy link
Copy Markdown
Member

@lunkwill42 lunkwill42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me :)

If you did integrate @jorund1's code, however, you should credit him with a Co-authored-by signature.

johannaengland and others added 3 commits August 12, 2025 11:17
Co-authored-by: Jørund Hellebø <jorund.hellebo@sikt.no>
Co-authored-by: Jørund Hellebø <jorund.hellebo@sikt.no>
@johannaengland johannaengland force-pushed the feature/bulk-download-qr-codes branch from 14681ed to 8a5d8a4 Compare August 12, 2025 09:18
@sonarqubecloud
Copy link
Copy Markdown

@johannaengland johannaengland merged commit 13b62e5 into Uninett:master Aug 12, 2025
16 checks passed
@johannaengland johannaengland deleted the feature/bulk-download-qr-codes branch August 12, 2025 09:40
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.

3 participants