Skip to content

Conversation

@d4mr
Copy link
Contributor

@d4mr d4mr commented Jan 27, 2025

PR-Codex overview

This PR introduces error handling for invalid BigInt values and integrates authorization handling into transaction schemas and processing. It enhances type safety and validation for transactions, particularly with authorizationList.

Detailed summary

  • Added badBigIntError function for error handling in src/server/middleware/error.ts.
  • Updated InsertedTransaction type to include authorizationList in src/shared/utils/transaction/types.ts.
  • Introduced requiredBigInt function for validating BigInt values in src/shared/utils/primitive-types.ts.
  • Created authorizationSchema and authorizationListSchema in src/server/schemas/transaction/authorization.ts.
  • Implemented toParsedAuthorization function for parsing authorization data.
  • Modified sendTransaction handler to include authorizationList in src/server/routes/backend-wallet/send-transaction.ts.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@github-actions
Copy link

github-actions bot commented Feb 6, 2025

This PR is stale because it has been open for 7 days with no activity. Remove stale label or comment or this PR will be closed in 3 days.

@github-actions github-actions bot closed this Feb 9, 2025
@d4mr d4mr reopened this Feb 18, 2025
@zeet-co
Copy link

zeet-co bot commented Feb 18, 2025

We're building your pull request over on Zeet.
Click me for more info about your build and deployment.
Once built, this branch can be tested at: https://tw-unreal-demo-engine-vwn0-production.engine-aws-usw2.zeet.app before merging 😉

@github-actions
Copy link

This PR is stale because it has been open for 7 days with no activity. Remove stale label or comment or this PR will be closed in 3 days.

@github-actions
Copy link

github-actions bot commented Mar 6, 2025

This PR is stale because it has been open for 7 days with no activity. Remove stale label or comment or this PR will be closed in 3 days.

@d4mr d4mr merged commit a2eca4e into main Mar 18, 2025
6 of 7 checks passed
@d4mr d4mr deleted the pb/eip-7702 branch March 18, 2025 20:16
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.

4 participants