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

feat: show qr-code with api key #523

Merged
merged 3 commits into from
Feb 18, 2022

Conversation

pedrolamas
Copy link
Member

@pedrolamas pedrolamas commented Feb 16, 2022

Fixes #411

Adds new dependency to qrcode.vue

image

Signed-off-by: Pedro Lamas [email protected]

@pedrolamas pedrolamas changed the title feat: show qr-code with api key (#514) feat: show qr-code with api key Feb 17, 2022
@pedrolamas pedrolamas added the FR - Enhancement New feature or request label Feb 17, 2022
src/components/ui/AppQrCode.vue Outdated Show resolved Hide resolved
src/components/ui/AppQrCode.vue Show resolved Hide resolved
@pedrolamas
Copy link
Member Author

@matmen thanks for reviewing, I've updated the PR following up your on comments (also updated the main image on the PR description to show the latest changes)

@pedrolamas pedrolamas added the UI - QoL Improvements to the UI label Feb 17, 2022
Copy link
Member

@matmen matmen left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@eduncan911 eduncan911 left a comment

Choose a reason for hiding this comment

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

Confirmed the image in the PR works on GitHub dark background on Android.

LGTM

@eduncan911 eduncan911 merged commit 6eab704 into fluidd-core:develop Feb 18, 2022
@pedrolamas pedrolamas deleted the pedrolamas/411 branch February 18, 2022 11:41
@Clon1998
Copy link

Clon1998 commented Feb 19, 2022

Greate work!
Thanks, this will make setting up a printer in my app way easier :)
Also can confirm that the QR code can be picked up trough my app's QR reader.

@matmen matmen added this to the Release: 1.17 milestone Feb 22, 2022
matmen pushed a commit to matmen/fluidd that referenced this pull request Jun 27, 2022
* feat: show qr-code with api key

Fixes fluidd-core#411

Signed-off-by: Pedro Lamas <[email protected]>

* fix: adds white border around qr-code

Signed-off-by: Pedro Lamas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FR - Enhancement New feature or request UI - QoL Improvements to the UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FR: QR-Code for API Key
4 participants