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

Feature/sign up #345

Merged
merged 4 commits into from
Nov 19, 2023
Merged

Feature/sign up #345

merged 4 commits into from
Nov 19, 2023

Conversation

Abhiek187
Copy link
Owner

I've now enabled users to create an account:

Screenshot 2023-11-19 175820 Screenshot 2023-11-19 175848

And I provided the option to delete said account. The user must type their email to confirm if they want to delete their account, a similar psychological tactic used in How to Stock.

Screenshot 2023-11-19 151900

I added the profile and aws.cognito.signin.user.admin scopes to allow these operations. And although it's possible to call DeleteUser via a regular API call, I opted to include the Cognito SDK since it's not well-documented. This doubled the build size, so I utilized dynamic imports to chunk the build since it's only used when deleting the account.

@Abhiek187 Abhiek187 added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code javascript Pull requests that update Javascript code labels Nov 19, 2023
@Abhiek187 Abhiek187 self-assigned this Nov 19, 2023
@Abhiek187 Abhiek187 merged commit 54fe595 into main Nov 19, 2023
13 checks passed
@Abhiek187 Abhiek187 deleted the feature/sign-up branch November 19, 2023 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation github_actions Pull requests that update GitHub Actions code javascript Pull requests that update Javascript code
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant