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

Possibly add something like onRenderError() or onError() #7

Open
theodorDiaconu opened this issue Jun 22, 2018 · 1 comment
Open

Possibly add something like onRenderError() or onError() #7

theodorDiaconu opened this issue Jun 22, 2018 · 1 comment
Labels
enhancement New feature or request

Comments

@theodorDiaconu
Copy link
Contributor

Errors can happen when we emit events.
We can add a handle to prevent that, and maybe have a sort of error component that is rendered
Errors can happen when we render improper data.

Questions:

  • Should we allow to simply override the whole rendering paradigm of the molecule if an error happens ?
@theodorDiaconu theodorDiaconu added the enhancement New feature or request label Jun 22, 2018
@StorytellerCZ
Copy link

That's a tough question. In my projects I have ErrorBoundary component which I wrap around components that might throw and then display a custom notification if the error is triggered. Might not be a bad option to add an option to show a component or text when error happens, but I would assume that every project is going to have their own error handling component which they are going to wrap around molecules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants