Sentry.ErrorBoundary showDialog is shown multiple times

[AbhiPrasad]

Discussed in #5629

^{Originally posted by mjooms August 24, 2022}
I was testing how the feedback dialog for ErrorBoundary works in a nextjs app. I used the showDialog prop. On a list of 14 items, I was throwing an error for each item (simulating by calling a non existent function).

To my surprise ErrorBoundary showed the feedback dialog 14 times... I had to click close 14 times in the feedback dialog to get rid of them all.

Is this the expected behavior?

[mydea]

cc @lforst is this something we should look into?

[lforst]

What is the correct behaviour we want here? Group the errors? Only take the last one.

[AbhiPrasad]

Ideally we can group the errors together, but this might not be possible. Maybe we don't allow for the model to be opened up multiple times? Can we check to only add the model if there is not one rendered?

[lforst]

Honestly, this might be a suboptimal configuration of the errorboundary with the showDialog option. I guess users should hoist the error boundary high enough on the tree so that only a sane number of errors bubble up to it. If you're wrapping, idk, individual list items with it, you're holding it wrong.