Skip to content

chore(passport): improve passport post to relayer error message#2658

Merged
nattb8 merged 1 commit into
mainfrom
chore/post-to-relayer-logs
Jul 7, 2025
Merged

chore(passport): improve passport post to relayer error message#2658
nattb8 merged 1 commit into
mainfrom
chore/post-to-relayer-logs

Conversation

@nattb8
Copy link
Copy Markdown
Contributor

@nattb8 nattb8 commented Jul 6, 2025

Hi👋, please ensure the PR title follows the below standards:

  • PR is titled with conventional commit style naming: type(scope): message. For example: feat(passport): my new feature
  • If you have introduced modification that necessitates immediate adjustments by this SDK's users to their applications, clients, or integrations to avert disruptions to existing features or functionalities, add a ! after the type(scope), for example feat(passport)!: my new breaking feature

Summary

Improved JSON parsing error message in postToRelayer to address SyntaxError: Unexpected end of JSON.

ID-3834

Detail and impact of the change

Added

Changed

Deprecated

Removed

Fixed

Security

Anything else worth calling out?

@nattb8 nattb8 requested a review from a team as a code owner July 6, 2025 23:27
@nattb8 nattb8 changed the title chore: improve passport post to relayer error message chore(passport): improve passport post to relayer error message Jul 6, 2025
@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Jul 6, 2025

View your CI Pipeline Execution ↗ for commit 418efe6

Command Status Duration Result
nx affected -t build,lint,test ✅ Succeeded 3m 10s View ↗
nx run-many -p @imtbl/sdk,@imtbl/checkout-widge... ✅ Succeeded 1m 20s View ↗

☁️ Nx Cloud last updated this comment at 2025-07-07 00:32:46 UTC

@nattb8 nattb8 force-pushed the chore/post-to-relayer-logs branch 2 times, most recently from f4a06fd to bc79181 Compare July 7, 2025 00:02
Comment thread packages/passport/sdk/src/zkEvm/relayerClient.ts Outdated
@nattb8 nattb8 force-pushed the chore/post-to-relayer-logs branch from bc79181 to 3dd95e2 Compare July 7, 2025 00:07
Comment thread packages/passport/sdk/src/zkEvm/relayerClient.ts Outdated
@nattb8 nattb8 force-pushed the chore/post-to-relayer-logs branch from 3dd95e2 to 5c8a66d Compare July 7, 2025 00:12
@nattb8 nattb8 force-pushed the chore/post-to-relayer-logs branch from 5c8a66d to 418efe6 Compare July 7, 2025 00:14
@nattb8 nattb8 requested a review from haydenfowler July 7, 2025 00:24
@nattb8 nattb8 added this pull request to the merge queue Jul 7, 2025
Merged via the queue into main with commit 3e1336d Jul 7, 2025
8 checks passed
@nattb8 nattb8 deleted the chore/post-to-relayer-logs branch July 7, 2025 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants