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

Document error boundary behavior #123

Open
MattDodsonEnglish opened this issue May 23, 2024 · 1 comment
Open

Document error boundary behavior #123

MattDodsonEnglish opened this issue May 23, 2024 · 1 comment
Labels
content needed plans to fill in content gaps

Comments

@MattDodsonEnglish
Copy link
Collaborator

MattDodsonEnglish commented May 23, 2024

Jarrah's review in #121 revealed there are a number of particulars regarding error boundaries. These should be documented.

Besides behavior, we also need to document what type of GO land errors would result in the event. Probably could be added to the reference.

@Jarrah-libremfg
Copy link
Contributor

Besides behavior, we also need to document what type of GO land errors would result in the event. Probably could be added to the reference.

Any Go error returned from a service task or call activity will result in an error boundary event being triggered if attached to the node. There's absolutely no filtering, so long as it's one of those nodes.

Probably also worth noting that you could attach an error boundary to a non-service task and the UI would look functional, but it would never be called.

MattDodsonEnglish added a commit that referenced this issue May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content needed plans to fill in content gaps
Projects
None yet
Development

No branches or pull requests

2 participants