Pr/error logging - Githubissues

epicweb-dev / epic-stack

This is a Full Stack app starter with the foundational things setup and configured for you to hit the ground running on your next EPIC idea.

https://www.epicweb.dev/epic-stack

MIT License

4.32k stars 355 forks source link

Pr/error logging #736

Closed zwhitchcox closed 3 months ago

zwhitchcox commented 3 months ago

Fixes bug in #735, where errors would get logged 2X when the error originated in the client code.

I've removed the duplicate error logging, so now, all errors are logged, and they are only logged one time.

Test Plan

This was manually tested.

Checklist

~[ ] Tests updated~ -- additional tests unnecessary, as it's just logging to the console
~[ ] Docs updated~ -- updating docs unnecessary, as this will be the expected behavior

Screenshots

Before, error thrown within React code is logged 2X:

Screenshot 2024-05-18 at 1 44 48 PM

After, they're logged only once:

Screenshot 2024-05-18 at 1 45 21 PM