Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce protocol version 3 #4741

Merged
merged 16 commits into from
Jan 15, 2025
Merged

Conversation

alexsporn
Copy link
Member

Description of change

Feature branch to introduce protocol version 3

valeriyr and others added 3 commits January 13, 2025 04:41
* fix(iota-framework): fixed `CoinManager` metadata immutability validation

* feat: create the protocol version `3`

* fix(iota-framework): `CoinManager` maximum supply check is fixed

* fix(iota-framework-snapshot): the snapshot is updated

* fix: update test snapshots

* fix(iota-framework): removed extra assert from the `CoinManager::add_additional_metadata` function

* fix(iota-framework-snapshot): the snapshot is updated

* fix: update test snapshots

* fix(iota-json-rpc-tests): changed the coin package serialization in `create_and_mint_coins`

* fix(iota-graphql-e2e-tests): the snapshots update
#4759)

* feat(iota-framework): added clock-based methods to the timelock module

* fix(iota-framework-snapshot): update the snapshot

* fix: update test snapshots

* fix(iota-graphql-e2e-tests): the snapshots update
… relocation (#19672)` (#4712)

* Merge from sui@59f115b

* Update snapshots and the generated files

* Set relocate_event_module config in version 3

* Remove the new config from old snapshots

* Update the openrpc.json

* Add regression test

* Convert the modification for version_2.snap

* Add protocol config change record

* Fix typo

* Fmt
@alexsporn alexsporn requested a review from a team as a code owner January 14, 2025 14:56
Copy link

vercel bot commented Jan 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
apps-backend ⬜️ Ignored (Inspect) Visit Preview Jan 15, 2025 2:05pm
apps-ui-kit ⬜️ Ignored (Inspect) Visit Preview Jan 15, 2025 2:05pm
rebased-explorer ⬜️ Ignored (Inspect) Visit Preview Jan 15, 2025 2:05pm
wallet-dashboard ⬜️ Ignored (Inspect) Visit Preview Jan 15, 2025 2:05pm

valeriyr and others added 5 commits January 14, 2025 16:53
…st fix (#4769)

* fix(iota-framework): the `test_expiration_time_is_passed` timelock test fix

* fix(iota-framework): added a `test_unlock_with_clock_expiration_time_is_passed` test
Copy link
Contributor

@kodemartin kodemartin left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@thibault-martinez thibault-martinez left a comment

Choose a reason for hiding this comment

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

LGTM apart from the fmt issue

@DaughterOfMars DaughterOfMars changed the title Introduce procotol version 3 Introduce protocol version 3 Jan 15, 2025
Co-authored-by: Levente Pap <levente.pap@iota.org>
@alexsporn alexsporn force-pushed the core-protocol/procotol-version-3 branch from 604ab48 to 6135a14 Compare January 15, 2025 14:04
/// The error returned if you try to edit immutable metadata
const ENoMutableMetadata: u64 = 5;
/// The maximum supply supported by `CoinManager`
const MAX_SUPPLY: u64 = 18_446_744_073_709_551_614u64;
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a reason this decreased by 1 from the number used below?

Copy link
Member Author

Choose a reason for hiding this comment

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

See #4595

@alexsporn alexsporn merged commit 5dd0863 into develop Jan 15, 2025
47 checks passed
@alexsporn alexsporn deleted the core-protocol/procotol-version-3 branch January 15, 2025 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core-protocol node Issues related to the Core Node team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants