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

Sign in via email/password with email verification roughly working. #28

Merged
merged 1 commit into from
Feb 15, 2025

Conversation

SailingSteve
Copy link
Member

@SailingSteve SailingSteve commented Feb 15, 2025

Apis are not yet protected from access without cookie/sessionID
Didn't want to get too diverged from other development.
Put most Routes in App behind the ProtectRoute -- If that causes problems in testing, you can disable it by taking Routes out of ProtectedRoutes.
Cleared some lint warnings and console errors in PersonSummaryHeader and PersonSummaryRow -- the boolean variables do get through to the StyledComponent and change the styles.
Added a sign in/out button to the HeaderBar.
Added VerifySecretCodeModal with styling same as in the WebApp.
Commented out more code we won't be using, to make searching easier.
Cleared more lint warnings.

Apis are not yet protected from access without cookie/sessionID
Didn't want to get too diverged from other development.
Put most Routes in App behind the ProtectRoute -- If that causes problems in testing, you can disable it by taking
Routes out of ProtectedRoutes.
Cleared some lint warnings and console errors in PersonSummaryHeader and PersonSummaryRow -- the boolean variables do get through to the StyledComponent and change the styles.
Added a sign in/out buttonm to the HeaderBar.
Added VerifySecretCodeModal with styling same as in the WebApp.
Commented out more code we won't be using to make searching easier.  Cleared more lint warnings.
@DaleMcGrew
Copy link
Member

Thank you @SailingSteve this is a lot of work! I will try it now. 👍

@DaleMcGrew DaleMcGrew merged commit b568311 into wevote:develop Feb 15, 2025
0 of 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.

2 participants