Skip to content

Add QR Code link to 'My Stuff'#2897

Merged
johannaengland merged 2 commits intoUninett:masterfrom
johannaengland:feature/qr-button-my-stuff
Aug 12, 2025
Merged

Add QR Code link to 'My Stuff'#2897
johannaengland merged 2 commits intoUninett:masterfrom
johannaengland:feature/qr-button-my-stuff

Conversation

@johannaengland
Copy link
Copy Markdown
Contributor

@johannaengland johannaengland commented May 2, 2024

Dependent on #3386. Another part to #2586.

This adds a button to 'My Stuff' that when clicked opens a modal displaying a QR code with the link to current page. This modal can be dismissed by clicking on the "X" button or clicking outside of the modal.

Screenshots - before and after

Before:

Screenshot from 2024-05-16 13-03-51

After:

image

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 2, 2024

Test results

     12 files       12 suites   11m 30s ⏱️
3 327 tests 3 327 ✔️ 0 💤 0
9 456 runs  9 456 ✔️ 0 💤 0

Results for commit f7aedce.

♻️ This comment has been updated with latest results.

@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch from b46efd6 to cda6a53 Compare May 2, 2024 13:30
@johannaengland johannaengland marked this pull request as draft May 2, 2024 13:30
@codecov
Copy link
Copy Markdown

codecov bot commented May 2, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.84%. Comparing base (36442ff) to head (23b7cff).
⚠️ Report is 526 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2897      +/-   ##
==========================================
- Coverage   60.87%   60.84%   -0.03%     
==========================================
  Files         607      607              
  Lines       43953    43953              
  Branches       48       48              
==========================================
- Hits        26758    26745      -13     
- Misses      17183    17196      +13     
  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/qr-button-my-stuff branch 3 times, most recently from 02aea42 to 459cce4 Compare May 6, 2024 13:49
@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch 2 times, most recently from 10b84e2 to f7aedce Compare May 7, 2024 09:09
@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch from f7aedce to 667c376 Compare May 16, 2024 07:29
@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch from a5951ec to 667c376 Compare May 16, 2024 11:30
@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch from 667c376 to 03eb2d7 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.

@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch from 03eb2d7 to a0b810a Compare August 7, 2024 16:18
@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch from a0b810a to 6f988cc Compare March 7, 2025 14:13
@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch from 6f988cc to f04382a Compare March 10, 2025 07:14
@sonarqubecloud
Copy link
Copy Markdown

@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch 2 times, most recently from 8d91221 to 691c63d Compare July 22, 2025 13:02
@johannaengland johannaengland marked this pull request as ready for review July 22, 2025 13:06
@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch from 691c63d to b8fd8e5 Compare July 22, 2025 13:17
@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch 3 times, most recently from 8626c71 to 5cd052b Compare July 28, 2025 08:34
@johannaengland johannaengland requested review from a team and removed request for hmpf, lunkwill42 and stveit July 28, 2025 08:35
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.

🤩

@johannaengland
Copy link
Copy Markdown
Contributor Author

I would also like to have a review from @podliashanyk before I merge this in

@johannaengland johannaengland marked this pull request as draft July 30, 2025 12:06
@johannaengland johannaengland marked this pull request as ready for review July 31, 2025 12:50
@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch 2 times, most recently from 171c298 to 6e886ce Compare August 5, 2025 07:18
@johannaengland johannaengland requested a review from a team August 5, 2025 09:07
@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch from 6e886ce to 3338dd3 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.

As long as @podliashanyk is satisfied with the CSS changes, so am I :)

Copy link
Copy Markdown
Contributor

@Simrayz Simrayz left a comment

Choose a reason for hiding this comment

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

Nod LGTM

Clicking the link will open a modal with a QR code to the current page
@johannaengland johannaengland force-pushed the feature/qr-button-my-stuff branch from 3338dd3 to 23b7cff Compare August 12, 2025 08:36
@sonarqubecloud
Copy link
Copy Markdown

@johannaengland johannaengland merged commit 4961212 into Uninett:master Aug 12, 2025
16 checks passed
@johannaengland johannaengland deleted the feature/qr-button-my-stuff branch August 12, 2025 09:16
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.

5 participants