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: add sign in button for auth providers w/ the only auth session request #6446
feat: add sign in button for auth providers w/ the only auth session request #6446
Conversation
Component tests have not been implemented yet |
packages/renderer/src/lib/preferences/PreferencesAuthenticationProvidersRendering.svelte
Outdated
Show resolved
Hide resolved
packages/renderer/src/lib/preferences/PreferencesAuthenticationProvidersRendering.svelte
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested successfully on my end. Thank you!
5a8cc57
to
0b355da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 works nicely on my machine
…request Signed-off-by: Denis Golovin <dgolovin@redhat.com>
…nProvidersRendering.svelte Co-authored-by: Florent BENOIT <fbenoit@redhat.com> Signed-off-by: Denis Golovin <dgolovin@users.noreply.github.com>
Signed-off-by: Denis Golovin <dgolovin@redhat.com>
0b355da
to
82c82a5
Compare
rebased to include the Tooltip changes from main branch |
Out of curiosity: what are these tooltip changes? |
@vrothberg #6445 |
Ready to merge? |
@vrothberg need @axel7083 or @jeffmaury review that were assigned for the review (two +1 from core team for new feat) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What does this PR do?
PR adds logic to auth provider record on Authentication settings page to show
Sign in
button in case of only one none silent authentication request registered. If there are multiple requests then Drop down menu is component is used to list all request for user selection.Screenshot / video of UI
What issues does this PR fix or reference?
How to test this PR?