-
Notifications
You must be signed in to change notification settings - Fork 43
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: implemented a reset feature when a user has selected a province #30
feat: implemented a reset feature when a user has selected a province #30
Conversation
@chaaals is attempting to deploy a commit to the OSSPH Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Nice, this is exactly what I was looking for. Thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, but I'll let a react expert do another review. Thanks.
@denzdelvillar @warengonzaga do you mind reviewing this? Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, good to have a custom message after reset.
I'm good with the current message in the success snackbar. Do you have something else in mind? |
No, I mean glad to have the message after reset. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks for this
Closes #25
Description
This PR adds a reset feature to the web app.
Implementation Details
I utilized context api in this change so that we can make
provinceLevels
a global state. I implemented thereset button
beside thesave image
button. I also implemented a constraint that users will only be able to reset when at least 1 province is selected. I also added a toast for better UX :))Demo
2023-04-27.15-00-15.mp4