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.

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

Improve front end error handling #5922

Merged
merged 1 commit into from
Jan 13, 2025
Merged

Improve front end error handling #5922

merged 1 commit into from
Jan 13, 2025

Conversation

mtsgrd
Copy link
Contributor

@mtsgrd mtsgrd commented Jan 10, 2025

  • handles signing error in one place

Copy link

vercel bot commented Jan 10, 2025

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

Name Status Preview Comments Updated (UTC)
gitbutler-components ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 13, 2025 11:29am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
gitbutler-web ⬜️ Skipped (Inspect) Jan 13, 2025 11:29am

- handles signing error in one place
@mtsgrd
Copy link
Contributor Author

mtsgrd commented Jan 13, 2025

@ndom91 this is just a small thing I did while working on other stuff.

Copy link
Contributor

@ndom91 ndom91 left a comment

Choose a reason for hiding this comment

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

Nice, knocked out like 5 TODOs here too 😂

Comment on lines 10 to 11
? true
: false;
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we can skip this two lines

typeof err.message === 'string' &&
'code' in err &&
typeof err.code === 'string') ||
false
Copy link
Contributor

Choose a reason for hiding this comment

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

We can also skip this line

apps/desktop/src/lib/error/typeguards.ts Show resolved Hide resolved
@vercel vercel bot temporarily deployed to Preview – gitbutler-web January 13, 2025 11:25 Inactive
@mtsgrd mtsgrd enabled auto-merge (rebase) January 13, 2025 11:26
@vercel vercel bot temporarily deployed to Preview – gitbutler-web January 13, 2025 11:28 Inactive
@mtsgrd mtsgrd merged commit 6307de9 into master Jan 13, 2025
17 checks passed
@mtsgrd mtsgrd deleted the frontend-errors branch January 13, 2025 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants