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 Microsoft login flow #1799

Merged
merged 4 commits into from Nov 8, 2023
Merged

Conversation

Trial97
Copy link
Member

@Trial97 Trial97 commented Nov 7, 2023

Stolen from TheKodeToad from here: https://discord.com/channels/1031648380885147709/1031823065937629267/1171229791513493544

Please don't hesitate to suggest any text changes.
I kept the old behavior in case the user browser was not open or something was unclear.

Also tested and it works.

Signed-off-by: Trial97 <alexandru.tripon97@gmail.com>
@Trial97 Trial97 added enhancement New feature or request simple change changelog:added A PR that appears under "Added" in the changelog labels Nov 7, 2023
@Trial97 Trial97 added this to the 8.1 milestone Nov 7, 2023
Copy link
Member

@TayouVR TayouVR left a comment

Choose a reason for hiding this comment

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

text could be a bit nicer, maybe like my suggestion below, but its fine either way (I want this entire dialog to get an overhaul some day x3)

launcher/ui/dialogs/MSALoginDialog.cpp Outdated Show resolved Hide resolved
Trial97 and others added 2 commits November 7, 2023 20:39
Co-authored-by: Tayou <git@tayou.org>
Signed-off-by: Alexandru Ionut Tripon <alexandru.tripon97@gmail.com>
Signed-off-by: Trial97 <alexandru.tripon97@gmail.com>
Copy link
Member

@getchoo getchoo left a comment

Choose a reason for hiding this comment

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

otherwise lgtm :)

launcher/ui/dialogs/MSALoginDialog.cpp Outdated Show resolved Hide resolved
Co-authored-by: seth <getchoo@tuta.io>
Signed-off-by: Alexandru Ionut Tripon <alexandru.tripon97@gmail.com>
@getchoo

This comment was marked as off-topic.

@Scrumplex Scrumplex merged commit 403c4ba into PrismLauncher:develop Nov 8, 2023
31 checks passed
@Scrumplex Scrumplex added changelog:changed A PR that appears under "Changed" in the changelog and removed changelog:added A PR that appears under "Added" in the changelog labels Nov 8, 2023
@Scrumplex Scrumplex changed the title simplify microsoft login Improve Microsoft login flow Nov 8, 2023
@TheKodeToad
Copy link
Member

wouldn't it be better to have a button instead of opening the browser straight away

@Trial97
Copy link
Member Author

Trial97 commented Nov 8, 2023

Not really as I want to simplify the process. not keep it the same.
Also a little late to the party as it was merged :(.
Feel free to open a PR where you can improve that behavior.

@TheKodeToad
Copy link
Member

ig it's fine

@Scrumplex Scrumplex added the manual backport PRs that have been backported manually label Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:changed A PR that appears under "Changed" in the changelog enhancement New feature or request manual backport PRs that have been backported manually simple change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants