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

Error message display. #5

Open
louh opened this issue Nov 27, 2017 · 0 comments
Open

Error message display. #5

louh opened this issue Nov 27, 2017 · 0 comments

Comments

@louh
Copy link
Member

louh commented Nov 27, 2017

1. Store

  • Components that report errors should put them in the Redux store. The components do not necessarily have to know about the store. We could write a single module that exports a addError() function that wraps the functionality for adding data to the store, so that the Redux store dependency is not baked into every component.

2. View

  • The UI renders the error somewhere:
    • Message in the control panel?
    • Modal?
    • Toast?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant