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

feat: Auth final #985

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

feat: Auth final #985

wants to merge 19 commits into from

Conversation

ap0nia
Copy link
Collaborator

@ap0nia ap0nia commented May 20, 2024

Summary

  • Frontend implementation of auth.
  • Uses current DynamoDB without migrating to RDS yet.

Refinements

  • If user is not logged in, they should not be able to change visibility.
  • If user is not logged in, they're allowed to pick any user ID to save.
  • If user is is logged in, the user ids to save the schedule under is readonly.
  • Toggle visibility settings in account dialog.

Test Plan

Future Follow-Up

  • Migrate system to RDS.

@MinhxNguyen7
Copy link
Member

MinhxNguyen7 commented May 20, 2024

Is this done? Also, what do you mean by "excludes DB"? Do you mean it's already been implemented?

Also, can you specify what you expect to/not to work so that I can check it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants