-
Notifications
You must be signed in to change notification settings - Fork 31
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
ui: improve notification component to support multiple different messages #123
Comments
mvidalgarcia
added a commit
to mvidalgarcia/reana-ui
that referenced
this issue
Oct 1, 2020
partially addresses reanahub#123
mvidalgarcia
added a commit
to mvidalgarcia/reana-ui
that referenced
this issue
Oct 5, 2020
partially addresses reanahub#123
mvidalgarcia
added a commit
to mvidalgarcia/reana-ui
that referenced
this issue
Dec 18, 2020
partially addresses reanahub#123
mvidalgarcia
added a commit
to mvidalgarcia/reana-ui
that referenced
this issue
Jan 7, 2021
partially addresses reanahub#123
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Notification component can be improved to handle different error messages happening one after another in a short period of time. Also, it should still be able to handle errors coming from redux store as well as from other components.
Couple of nice suggestions regarding the implementation:
The text was updated successfully, but these errors were encountered: