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

[EPO-8615] Basic user flows are styled #68

Merged
merged 1 commit into from
Oct 31, 2023
Merged

[EPO-8615] Basic user flows are styled #68

merged 1 commit into from
Oct 31, 2023

Conversation

blnkt
Copy link

@blnkt blnkt commented Oct 26, 2023

JIRA: https://jira.lsstcorp.org/browse/EPO-8615

What this change does

Touches almost every existing User dialog. Only address visual errors, and some very minimal functional changes. DOes not include changes to the dialogs for password reset or account activation success or login success as those views need changes to the API first before they can be utilized.

Copy link

A preview of this PR will be available at https://epo8615-dot-investigations-client-dot-skyviewer.appspot.com until the request is closed.

Includes:
 - LocaleStrings
 - GoogleSSO buttons
 - FacebookSSO buttons
 - SignIn dialog
 - ForgetPassword dialog
 - SelectGroup & SElectProvider dialogs
 - SignUp dialog
 - Linting fixes

[F] LocaleSTrings for sign in messages

[B] Correct continue wo login button styles

[F] Facebook SSO redirects to exit page

[F] Sign in dialog styled

[F] SelectGroup is styled

[F] ForgetPassword is styled

[F] SelectProvider is styled

[F] SignUp is styled

[F] SignUp is styled

[B] Fix linting errors
@blnkt blnkt marked this pull request as ready for review October 31, 2023 17:42
Copy link

A preview of this PR will be available at https://epo8615-dot-investigations-client-dot-skyviewer.appspot.com until the request is closed.

@blnkt blnkt changed the title Epo 8615 [EPO-8615] Basic user flows are styled Oct 31, 2023
@blnkt blnkt merged commit aa8d83e into develop Oct 31, 2023
2 checks passed
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.

1 participant