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.

Sign up for GitHub

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

Jump to bottom

fix: Surface all exceptions to the user instead of crashing #565

Merged
merged 1 commit into from
Mar 24, 2024
Merged

fix: Surface all exceptions to the user instead of crashing #565

merged 1 commit into from
Mar 24, 2024

Commits on Mar 24, 2024

  1. fix: Surface all exceptions to the user instead of crashing 

    Previous code would handle some expected exceptions (IO, HTTP) when
fetching a timeline, and show them to the user. Any other exception
would crash.

Now, surface all exceptions. Treat IO and HTTP exceptions as retryable
and show the "Retry" option, all others are considered non-retryable.

Provide a specific error string for exceptions caused by bad JSON.
    @nikclayton
    nikclayton committed Mar 24, 2024
    Configuration menu
    Copy the full SHA
    ef22350 View commit details
    Browse the repository at this point in the history