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

Add ErrorBoundary around Rules panel #9733

Merged
merged 2 commits into from
Sep 14, 2023
Merged

Add ErrorBoundary around Rules panel #9733

merged 2 commits into from
Sep 14, 2023

Conversation

bvaughn
Copy link
Contributor

@bvaughn bvaughn commented Sep 14, 2023

Loom overview

While I was in there I also removed the unused react-error-boundary dependency.

@vercel
Copy link

vercel bot commented Sep 14, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
devtools ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 14, 2023 6:31pm

@replay-io
Copy link

replay-io bot commented Sep 14, 2023

E2E Tests

80 replays were recorded for 7522c00.

image 1 Failed
image 79 Passed

View test run on Replay ↗︎

Snapshot Tests

100 replays were recorded for 7522c00.

image 0 Failed
image 100 Passed

View test run on Replay ↗︎

@@ -14038,17 +14037,6 @@ __metadata:
languageName: node
linkType: hard

"playwright@npm:^1.37.1":
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Did someone not run yarn install after updating package.json or is Yarn just being clowny?

Copy link
Collaborator

Choose a reason for hiding this comment

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

That one's probably my fault from yesterday, sorry!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No worries!

@socket-security
Copy link

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: playwright@1.37.1, react-error-boundary@4.0.11

@bvaughn bvaughn merged commit 2804747 into main Sep 14, 2023
33 of 35 checks passed
@bvaughn bvaughn deleted the FE-1907 branch September 14, 2023 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants