Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Don't stop execution if error position function fails #216

Merged
merged 1 commit into from
Dec 19, 2023

Conversation

maxwofford
Copy link
Member

This is to address #214. As mentioned in this comment #214 (comment), this PR doesn't fully fix #214. Rather, it adds a sane default to not crash the page's execution if getPosFromError fails to find where an error is.

Copy link

vercel bot commented Dec 19, 2023

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

Name Status Preview Updated (UTC)
blot ❌ Failed (Inspect) Dec 19, 2023 5:54am
blot-3dgallery ✅ Ready (Inspect) Visit Preview Dec 19, 2023 5:54am

@maxwofford
Copy link
Member Author

This build will fail until #210 is merged

@leomcelroy leomcelroy merged commit 56f405d into main Dec 19, 2023
1 check passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

console.{log|warn|error} fails on non-chrome browsers
2 participants