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

Stay Logged in Checkbox #84

Merged
merged 15 commits into from
Feb 19, 2025
Merged

Stay Logged in Checkbox #84

merged 15 commits into from
Feb 19, 2025

Conversation

JustinTan-1
Copy link
Contributor

@JustinTan-1 JustinTan-1 commented Feb 12, 2025

Resolves #73

Add stay logged in Checkbox to authentication page

What was edited, added, and or created

  • Add new Checkbox component
  • StaySignedIn State to authentication page
  • Modal appears when user checks box

Screenshot/Video 📷

RPReplay_Final1739384578.MP4

Copy link
Collaborator

@xihai01 xihai01 left a comment

Choose a reason for hiding this comment

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

lgtm!

@7riumph
Copy link
Member

7riumph commented Feb 18, 2025

Thanks for the work on this @JustinTan-1.

Address @xihai01's comments, pull down the recent changes, then we'll approve and merge.

In the future...

  • Work on a custom feature/issue_number or appropriately named branch instead of directly on your main
  • Add jest test(s)

@JustinTan-1
Copy link
Contributor Author

Alright, will do for next time. Addressed comments, let me know if anything else needs to be changed

@7riumph
Copy link
Member

7riumph commented Feb 19, 2025

Alright, will do for next time. Addressed comments, let me know if anything else needs to be changed

Nice, pull down the recent changes/update your fork's branch so the merge conflict goes away (Press the Sync fork button in github)

Copy link
Member

@7riumph 7riumph left a comment

Choose a reason for hiding this comment

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

Great work on this :)

@7riumph
Copy link
Member

7riumph commented Feb 19, 2025

@JustinTan-1 Also let me know whenever you've forked CASA and finished their Ruby on Rails setup process

Next task is here, can answer questions once you're set.

@7riumph 7riumph merged commit 7a88f21 into ctc-casa-ios:main Feb 19, 2025
3 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.

Add "Stay Logged In" checkbox to login page and alert users that session will be valid until they sign-out
3 participants