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

[candidate_profile] Add error boundary to cards with errors #9333

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

driusan
Copy link
Collaborator

@driusan driusan commented Sep 11, 2024

This adds an error boundary to the jsx/Card class, so that when a widget doesn't render properly on the candidate profile page that panel is displayed with an error and the rendering of other widgets is unaffected, instead of crashing and showing a blank page.

Fixes an issue I can't find keywords to find the reference for.

@driusan driusan added the Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) label Sep 11, 2024
This adds an error boundary to the jsx/Card class, so that when
a widget doesn't render properly on the candidate profile page that
widget is displayed with an error and the rendering of other widgets
is unaffected, instead of showing a blank page.
@cmadjar
Copy link
Collaborator

cmadjar commented Oct 1, 2024

@GeorgeMurad could you review and test this PR?

Copy link
Contributor

@GeorgeMurad GeorgeMurad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@driusan driusan merged commit b746ea4 into aces:main Oct 7, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants