Skip to content

Conversation

@MitchDickinson
Copy link
Contributor

@MitchDickinson MitchDickinson commented Oct 29, 2025

Add error handling principles documentation starting point

Copy link
Contributor Author

MitchDickinson commented Oct 29, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@MitchDickinson MitchDickinson marked this pull request as ready for review October 29, 2025 11:53
@MitchDickinson MitchDickinson requested a review from a team as a code owner October 29, 2025 11:53
Copy link
Contributor

@gonzaloriestra gonzaloriestra left a comment

Choose a reason for hiding this comment

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

Nice addition, thanks!

No need to add it now, but more stuff we should mention:

  • Error handling on API requests: automatic retries, token refresh and so on
  • How to use the Error classes: message, tryMessage, customSections, the tokenItem type and so on (partly explained here)

@MitchDickinson MitchDickinson force-pushed the error_handling_principles branch from 79f75ea to 6ade1ee Compare November 5, 2025 15:21
@MitchDickinson
Copy link
Contributor Author

Gonna merge cuz I want to link this in a few places - we can always continue iterating in follow-ups

@MitchDickinson MitchDickinson force-pushed the error_handling_principles branch from 6ade1ee to 236e28e Compare November 6, 2025 14:18
@MitchDickinson MitchDickinson merged commit 15812aa into main Nov 6, 2025
4 checks passed
@MitchDickinson MitchDickinson deleted the error_handling_principles branch November 6, 2025 15:31
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.

3 participants