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

♻️ (signer-solana) [NO-ISSUE]: Improve error handling #626

Merged

Conversation

fAnselmi-Ledger
Copy link
Contributor

📝 Description

Improve error handling

❓ Context

  • JIRA or GitHub link: [NO-ISSUE]

✅ Checklist

Pull Requests must pass CI checks and undergo code review. Set the PR as Draft if it is not yet ready for review.

  • Covered by automatic tests
  • Changeset is provided
  • Impact of the changes:
    • list of the changes

🧐 Checklist for the PR Reviewers

  • The code aligns with the requirements described in the linked JIRA or GitHub issue.
  • The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
  • There are no undocumented trade-offs, technical debt, or maintainability issues.
  • The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
  • Any new dependencies have been justified and documented.

Copy link

vercel bot commented Jan 23, 2025

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

Name Status Preview Comments Updated (UTC)
device-sdk-ts-sample ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 23, 2025 4:18pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
doc-device-management-kit ⬜️ Ignored (Inspect) Visit Preview Jan 23, 2025 4:18pm

Copy link
Contributor

github-actions bot commented Jan 23, 2025

Messages
Danger: All checks passed successfully! 🎉

Generated by 🚫 dangerJS against 22d1ec0

@fAnselmi-Ledger fAnselmi-Ledger force-pushed the chore/no-issue-refactor-solana-error-handling branch from adae7d7 to fa7f949 Compare January 23, 2025 09:01
@fAnselmi-Ledger fAnselmi-Ledger force-pushed the chore/no-issue-refactor-solana-error-handling branch from fa7f949 to 4ae8617 Compare January 23, 2025 09:04
@fAnselmi-Ledger fAnselmi-Ledger changed the title ♻️ (signer-solana): Improve error handling ♻️ (signer-solana) [NO-ISSUE]: Improve error handling Jan 23, 2025
@fAnselmi-Ledger fAnselmi-Ledger force-pushed the chore/no-issue-refactor-solana-error-handling branch from 4ae8617 to f98a6c9 Compare January 23, 2025 15:09
@fAnselmi-Ledger fAnselmi-Ledger marked this pull request as ready for review January 23, 2025 15:09
@fAnselmi-Ledger fAnselmi-Ledger requested a review from a team as a code owner January 23, 2025 15:09
@fAnselmi-Ledger fAnselmi-Ledger force-pushed the chore/no-issue-refactor-solana-error-handling branch from f98a6c9 to 551fd56 Compare January 23, 2025 15:11
@fAnselmi-Ledger fAnselmi-Ledger added this pull request to the merge queue Jan 23, 2025
@fAnselmi-Ledger fAnselmi-Ledger removed this pull request from the merge queue due to a manual request Jan 23, 2025
@fAnselmi-Ledger fAnselmi-Ledger added this pull request to the merge queue Jan 23, 2025
Merged via the queue into develop with commit ac3edef Jan 23, 2025
10 checks passed
@fAnselmi-Ledger fAnselmi-Ledger deleted the chore/no-issue-refactor-solana-error-handling branch January 23, 2025 16:44
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