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

[custom] Fix and Refactor react-error boundary #340

Merged
merged 8 commits into from Jan 22, 2024

Conversation

sudhanshutech
Copy link
Member

@sudhanshutech sudhanshutech commented Nov 15, 2023

Notes for Reviewers

This PR fixes #223

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Sudhanshu Dasgupta <dasguptashivam23@gmail.com>
@sudhanshutech sudhanshutech added pr/on hold PR/Issue on hold and removed component/components labels Nov 15, 2023
@sudhanshutech sudhanshutech changed the title fix(refactor): refactor error boundary [custom] Refactor error boundary Nov 15, 2023
Signed-off-by: Sudhanshu Dasgupta <dasguptashivam23@gmail.com>
@sudhanshutech sudhanshutech changed the title [custom] Refactor error boundary [custom] Fix and Refactor error boundary Nov 15, 2023
@nebula-aac nebula-aac requested a review from a team November 15, 2023 15:13
@sudhanshutech
Copy link
Member Author

@nebula-aac should we also need to move this to root

@leecalcote
Copy link
Member

@sudhanshutech there is new CSS for the MUIcheckbox's in Cloud that needs to be migrated to Sistent...

@nebula-aac
Copy link
Contributor

@sudhanshutech can you update your PR with the latest upstream, I'll test this afterwards

@sudhanshutech sudhanshutech changed the title [custom] Fix and Refactor error boundary [custom] Fix and Refactor react-error boundary Nov 28, 2023
Copy link
Contributor

@aabidsofi19 aabidsofi19 left a comment

Choose a reason for hiding this comment

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

As we dont need to make react-error-boundary external . we should remove the package when we use it from sistent

@sudhanshutech sudhanshutech removed the pr/on hold PR/Issue on hold label Dec 6, 2023
@Yashsharma1911
Copy link
Member

@sudhanshutech conflicts, can you update this error boundary component with support button you can copy component that we have in MeshMap

@sudhanshutech
Copy link
Member Author

ok, are you referring for rjsf component? @Yashsharma1911

Signed-off-by: Sudhanshu Dasgupta <dasguptashivam23@gmail.com>
Signed-off-by: Sudhanshu Dasgupta <dasguptashivam23@gmail.com>
Signed-off-by: Sudhanshu Dasgupta <dasguptashivam23@gmail.com>
Copy link
Contributor

@aabidsofi19 aabidsofi19 left a comment

Choose a reason for hiding this comment

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

Looks good

@Yashsharma1911 Yashsharma1911 merged commit 63c9e69 into layer5io:master Jan 22, 2024
8 checks passed
@Yashsharma1911 Yashsharma1911 mentioned this pull request Jan 23, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

[library]: create react-error-boundary library
5 participants