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

Improve login #464

Merged
merged 2 commits into from
Nov 17, 2023
Merged

Improve login #464

merged 2 commits into from
Nov 17, 2023

Conversation

Milo123459
Copy link
Collaborator

If you answer no to the prompt about opening the browser, instead of just exiting, it'll now perform a browserless login instead.

@Milo123459 Milo123459 added the release/patch Author patch release label Oct 26, 2023
@zuchka
Copy link
Contributor

zuchka commented Nov 17, 2023

@Milo123459 can you confirm that this is how your feature should work? I got this prompt and then I clicked out to the browser link. Just want to double check.

CleanShot 2023-11-17 at 12 19 11@2x

@Milo123459
Copy link
Collaborator Author

the wording is slightly confusing which could be changed in a follow up pr, but yes this is intended behaviour @zuchka

Copy link
Contributor

@zuchka zuchka left a comment

Choose a reason for hiding this comment

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

🚀

@Milo123459 Milo123459 merged commit 0bd1444 into master Nov 17, 2023
@Milo123459 Milo123459 deleted the milo/improve-login branch November 17, 2023 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/patch Author patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants