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.

Sign up for GitHub

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

Jump to bottom

Replace window alerts with React Notifications #469

Closed
epadams opened this issue Apr 1, 2024 · 0 comments · Fixed by #484
Closed

Replace window alerts with React Notifications #469

epadams opened this issue Apr 1, 2024 · 0 comments · Fixed by #484
Assignees
@epadams
Labels
enhancement New feature or request
Milestone
1.0 Release

Comments

@epadams
Copy link
Contributor

epadams commented Apr 1, 2024

We should look into replacing all window alerts with React notifications, it would look a lot cleaner. They're pretty simple to use and shouldn't take much time to convert what we have over.
@epadams epadams added the enhancement New feature or request label Apr 1, 2024
@epadams epadams changed the title Replact window alerts with React Notifications Replace window alerts with React Notifications Apr 1, 2024
@evan-scales evan-scales added this to the 1.0 Release milestone Apr 1, 2024
@epadams epadams linked a pull request Apr 7, 2024 that will close this issue
Convert relevant alerts to react notifications #484
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@epadams epadams

Labels
enhancement New feature or request
Projects
None yet
Milestone
1.0 Release
Development

Successfully merging a pull request may close this issue.

Convert relevant alerts to react notifications SCCapstone/PalmettoProgrammers
2 participants
@epadams @evan-scales