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

Add error/catch boundaries in Remix to deal with server-side errors #391

Closed
humphd opened this issue Mar 20, 2023 · 2 comments · Fixed by #554
Closed

Add error/catch boundaries in Remix to deal with server-side errors #391

humphd opened this issue Mar 20, 2023 · 2 comments · Fixed by #554
Assignees
Labels
category: front end Front end part of our web service

Comments

@humphd
Copy link
Contributor

humphd commented Mar 20, 2023

We have a bunch of places in the UI's server code where we throw an error or response. These are mostly "should never happen" type situations, but as we all know, stuff happens!

Remix provides a mechanism for the front-end to deal with these error cases and show components instead of just crashing the app:

There are lots of good blog posts about this, like this one.

Let's start by finding all of the errors we need to deal with, and then we can split up the work of creating UI to show the user when things happen.

cc @Eakam1007, @Ririio, @Myrfion

@humphd humphd added the category: front end Front end part of our web service label Mar 20, 2023
@humphd
Copy link
Contributor Author

humphd commented Mar 23, 2023

The 409 we return from app/routes/__index/dns-records/index.tsx is a good example of where we need error UI.

@humphd
Copy link
Contributor Author

humphd commented Apr 9, 2023

There are some basic UI templates we could use for this at https://chakra-templates.dev/components/result

@Myrfion Myrfion linked a pull request Apr 10, 2023 that will close this issue
@humphd humphd added this to the Milestone 1.0a milestone Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: front end Front end part of our web service
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants