Skip to content

Feat/sealed bid auction/712 signature fix#169

Merged
kss-t1 merged 6 commits into
canaryfrom
feat/sealed-bid-auction/712-signature-fix
Aug 26, 2025
Merged

Feat/sealed bid auction/712 signature fix#169
kss-t1 merged 6 commits into
canaryfrom
feat/sealed-bid-auction/712-signature-fix

Conversation

@kss-t1

@kss-t1 kss-t1 commented Aug 25, 2025

Copy link
Copy Markdown
Member

Description

Correctly creates EIP712 signature for Sealed Bid Auction result
Also adapts 7683 test scripts to Sealed Bid Auction App

Motivation and Context

How Has This Been Tested?

Types of changes (remove all unchecked types)

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Style (style only changes)
  • Docs
  • Refactor (code that does not add new functionality nor fixes a bug)

Checklist:

  • If my change requires a change to the documentation, I have updated the documentation accordingly, and in either case, have checked this box to attest to my assessment of this requirement with regard to my change.
  • If my change requires additions or updates to any deployment scripts to ensure that the protocol is functional (Makefile, Dockerfile, Forge Script, etc.), I have made these changes, and in either case, have checked this box to attest to my assessment of this requirement with regard to my change.
  • If my change requires additional test coverage, I have created those tests accordingly, and in either case, have checked this box to attest to my assessment of this requirement with regard to my change.

@kss-t1 kss-t1 requested a review from aharvet August 26, 2025 00:48
@kss-t1 kss-t1 merged commit 48c2536 into canary Aug 26, 2025
4 checks passed
@kss-t1 kss-t1 deleted the feat/sealed-bid-auction/712-signature-fix branch August 26, 2025 09:04
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.

2 participants