Skip to content
This repository has been archived by the owner on Oct 1, 2024. It is now read-only.

Pass caught server error to custom error component #1667

Merged
merged 2 commits into from
Nov 15, 2020

Conversation

beckygowland
Copy link
Contributor

@beckygowland beckygowland commented Nov 2, 2020

Description

For custom error components, there's the server error that is supposed to be passed in as a prop that's currently undefined at the moment. This sets that value to the same error as what's caught and logged in the react server.

Type of change

  • React-server Patch: Bug (non-breaking change which fixes an issue)

Checklist

  • I have added a changelog entry, prefixed by the type of change noted above (Documentation fix and Test update does not need a changelog as we do not publish new version)

Copy link
Contributor

@ismail-syed ismail-syed left a comment

Choose a reason for hiding this comment

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

👍 💯
Can you please open an issue for the bug we caught with client-side hydration issues? Thanks!

@beckygowland
Copy link
Contributor Author

beckygowland commented Nov 3, 2020

Can you please open an issue for the bug we caught with client-side hydration issues? Thanks!

🎉 opened #1666

@beckygowland beckygowland force-pushed the add-error-prop-to-custom-error branch 4 times, most recently from 84548d1 to 095d2a0 Compare November 5, 2020 15:30
@beckygowland beckygowland merged commit a7c79ce into master Nov 15, 2020
@beckygowland beckygowland deleted the add-error-prop-to-custom-error branch November 15, 2020 19:31
@alexandcote alexandcote temporarily deployed to production December 3, 2020 18:11 Inactive
@marutypes marutypes temporarily deployed to gem February 8, 2021 15:51 Inactive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants