Skip to content

Conversation

@saucepoint
Copy link
Collaborator

@saucepoint saucepoint commented Oct 23, 2025

DO NOT MERGE UNTIL:

  • Set the initial merkle root
  • Set the initial fee values by fee tier
  • Set the threshold
  • Set the number of tokens to release in ExchangeReleaser
  • Set the salts for vanity addresses
  • UNIVesting is added
  • Regenerate merkle root

@vercel
Copy link

vercel bot commented Oct 23, 2025

@saucepoint must be a member of the protocols-tool-test team on Vercel to deploy.
- Click here to add @saucepoint to the team.
- If you initiated this build, request access.

Learn more about collaboration on Vercel and other options here.

Copy link
Contributor

@marktoda marktoda left a comment

Choose a reason for hiding this comment

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

separately we should make a script that enacts the proposal so we can generate calldata and seatbelt

Copy link
Member

@snreynolds snreynolds left a comment

Choose a reason for hiding this comment

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

looks good just comment suggestions, and I will re review after we merge #100

@marktoda marktoda marked this pull request as ready for review November 21, 2025 20:18
@gretzke
Copy link
Collaborator

gretzke commented Nov 21, 2025

@wildmolasses please sign your commits so this can be merged! :)

Copy link
Collaborator Author

@saucepoint saucepoint left a comment

Choose a reason for hiding this comment

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

cant approve my own PR haha but agreed this ones getting quite big

Comment on lines +105 to +107
assertEq(
uniSupplyBefore - IERC20(RESOURCE).totalSupply(), THRESHOLD, "UNI not burned correctly"
);
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

seems like the OP-bridged tokens decrement totalSupply for transfers to the bridge? super cool

@marktoda marktoda merged commit d4f328e into main Nov 22, 2025
5 checks passed
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.

6 participants