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

Forward to web if device trust authorize cancelled #41079

Merged
merged 1 commit into from
May 1, 2024

Conversation

avatus
Copy link
Contributor

@avatus avatus commented Apr 30, 2024

If a user decides to not authorize their session, the cancel button will now send them to /web in the browser. This will allow the user to continue using the web UI but without device trust enabled (similar to selecting 'continue without device trust' in the passthrough page)

If a user decides to not authorize their session, the cancel button will
now send them to /web in the browser. This will allow the user to
continue using the web UI but without device trust enabled (similar to
selecting 'continue without device trust' in the passthrough page)
@avatus avatus added the no-changelog Indicates that a PR does not require a changelog entry label Apr 30, 2024
@avatus avatus requested review from ravicious and gzdunek April 30, 2024 22:02
@avatus
Copy link
Contributor Author

avatus commented Apr 30, 2024

fyi @codingllama

@github-actions github-actions bot requested a review from rudream April 30, 2024 22:03
@github-actions github-actions bot requested a review from ryanclark April 30, 2024 22:03
@github-actions github-actions bot added the ui label Apr 30, 2024
@avatus avatus added this pull request to the merge queue May 1, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 1, 2024
@avatus avatus added this pull request to the merge queue May 1, 2024
Merged via the queue into master with commit 5170df5 May 1, 2024
41 of 42 checks passed
@avatus avatus deleted the avatus/dtw_connect_cancel branch May 1, 2024 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Indicates that a PR does not require a changelog entry size/sm ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants