-
Notifications
You must be signed in to change notification settings - Fork 37
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
Add Confirmation Prompt for Cancelling and Deleting Student Questions #71
Conversation
Loving this refactoring work <3 will do a full review when I'm more awake tomorrow. In the meantime, want to fix the ESLint errors and write a changelog entry? It might make sense to move the |
dfbd39b
to
ba334f3
Compare
ba334f3
to
089bad7
Compare
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 great Aidan! Thanks for this 😄
CHANGELOG.md
Outdated
@@ -9,6 +9,7 @@ with the current date and the next changes should go under a **[Next]** header. | |||
|
|||
* Improve spacing of course buttons on homepage. ([@nwalters512](https://github.com/nwalters512) in [#60](https://github.com/illinois/queue/pull/60)) | |||
* Add support and tooling for Sequelize database migrations. ([@nwalters512](https://github.com/nwalters512) in [#56](https://github.com/illinois/queue/pull/56)) | |||
* Add Confirmation Prompt for Cancelling and Deleting Student Questions. ([@muakasan](https://github.com/muakasan) in [#71](https://github.com/illinois/queue/pull/71)) |
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.
Sentence case please 😛
Resolves #47