Skip to content

Conversation

@RodrigoAD
Copy link
Member

@RodrigoAD RodrigoAD commented Oct 22, 2025

Ownership Tests

  • LnR
  • BnM
  • Managed

Config tests (missing set_pool due to admin issue in contracts)

  • LnR
  • BnM
  • Managed

@RodrigoAD RodrigoAD force-pushed the token-pool-mcms-tests branch 2 times, most recently from 63f5a7c to 8dd51d1 Compare October 23, 2025 10:45
@RodrigoAD RodrigoAD marked this pull request as ready for review October 23, 2025 11:27
@RodrigoAD RodrigoAD requested a review from a team as a code owner October 23, 2025 11:27
const (
// DefaultGasBudget is the default gas budget for transactions
DefaultGasBudget uint64 = 500_000_000
DefaultGasBudget uint64 = 10_000_000_000
Copy link
Member Author

Choose a reason for hiding this comment

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

This will likely be used only by MCMS. I removed it from the lib so we control from Sui the default.
Other transactions can control the Budget from the Operations

@RodrigoAD RodrigoAD force-pushed the token-pool-mcms-tests branch 2 times, most recently from 6ad6c13 to 1f84058 Compare October 24, 2025 09:00
@RodrigoAD RodrigoAD requested a review from JohnChangUK October 24, 2025 14:22
@RodrigoAD RodrigoAD force-pushed the token-pool-mcms-tests branch from 947241f to 9f2ea44 Compare October 24, 2025 17:24
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.

1 participant