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

Display pres req link for debugging #419

Merged
merged 2 commits into from
Feb 15, 2024
Merged

Conversation

loneil
Copy link
Contributor

@loneil loneil commented Feb 15, 2024

(this is not related to BC Wallet deep link feature upcoming)

It's onerous to debug the QR code link on a VCAuth login. Have to snip code out, save, upload to a qr decode or something like that. Would be good to just see the pure text link that's converted to the QR Code.

Can use this to debug locally, or if needed do operational support for a user that's having an issue with the QR code.

Add an easter egg (BC Wallet style, tap 10 times on the BC Gov logo) that exposes the link in a textbox

image

@loneil loneil requested a review from esune February 15, 2024 18:30
Signed-off-by: Lucas ONeil <[email protected]>
@loneil loneil force-pushed the feature/presReqLinkDebugging branch from ee0fa86 to 7758c18 Compare February 15, 2024 18:31
@loneil loneil requested review from Jsyro and jamshale February 15, 2024 18:31
@swcurran
Copy link
Contributor

There is another issue (#413 ) that requires we add a deep link button to the VC-Authn-OIDC. That ticket should be a very high priority. This implements a lot of it and happy to merge it, but please take a look and carry out that issue.

@loneil
Copy link
Contributor Author

loneil commented Feb 15, 2024

#413 is in progress

@@ -88,7 +88,7 @@ export default function getRouter(basePath = '/') {
document.title = to.meta.title
? to.meta.title
: process.env.VUE_APP_TITLE;
} else document.title = 'Demo VC-Authn-OIDC App”'; // default title
Copy link
Contributor

Choose a reason for hiding this comment

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

Think I did this a long time ago :/

@esune esune merged commit 7d23d56 into main Feb 15, 2024
5 checks passed
@esune esune deleted the feature/presReqLinkDebugging branch February 15, 2024 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants