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

Implement error boundary #46

Closed
rmanaem opened this issue Feb 19, 2025 · 3 comments · Fixed by #47
Closed

Implement error boundary #46

rmanaem opened this issue Feb 19, 2025 · 3 comments · Fixed by #47
Assignees

Comments

@rmanaem
Copy link
Contributor

rmanaem commented Feb 19, 2025

No description provided.

@rmanaem rmanaem transferred this issue from neurobagel/planning Feb 19, 2025
@rmanaem rmanaem self-assigned this Feb 19, 2025
@rmanaem rmanaem moved this to Implement - Active in Neurobagel Feb 19, 2025
@rmanaem
Copy link
Contributor Author

rmanaem commented Feb 19, 2025

@alyssadai

Here is the screen shot of what the error boundary UI looks like
Where would you like to include the link to docs?
Also any feedback on the emoji?

Image

@surchs
Copy link

surchs commented Feb 19, 2025

I like it. I think the annotation tools is more targeted at grad students and early career researchers. A fun style like this is cool

@alyssadai
Copy link

Looking good! How about amending the message to something like:

... Please try again, check out the documentation, or open an issue.

?

@rmanaem rmanaem moved this from Implement - Active to Implement - Done in Neurobagel Feb 19, 2025
@surchs surchs moved this from Implement - Done to Review - Active in Neurobagel Feb 20, 2025
@github-project-automation github-project-automation bot moved this from Review - Active to Review - Done in Neurobagel Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Review - Done
Development

Successfully merging a pull request may close this issue.

3 participants