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

[framework] Adds TokenPolicyCreated event to the token module #15128

Merged
merged 4 commits into from
Dec 1, 2023

Conversation

damirka
Copy link
Contributor

@damirka damirka commented Nov 30, 2023

Description

Adds a simple event emitted when a Policy is shared (or frozen - in the future).

Test Plan

It's just an event.


If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.

Type of Change (Check all that apply)

  • protocol change
  • user-visible impact
  • breaking change for a client SDKs
  • breaking change for FNs (FN binary must upgrade)
  • breaking change for validators or node operators (must upgrade binaries)
  • breaking change for on-chain data layout
  • necessitate either a data wipe or data migration

Release notes

@damirka damirka self-assigned this Nov 30, 2023
Copy link

vercel bot commented Nov 30, 2023

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

Name Status Preview Comments Updated (UTC)
mysten-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 1, 2023 3:46pm
sui-core ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 1, 2023 3:46pm
sui-typescript-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 1, 2023 3:46pm
3 Ignored Deployments
Name Status Preview Comments Updated (UTC)
explorer ⬜️ Ignored (Inspect) Visit Preview Dec 1, 2023 3:46pm
multisig-toolkit ⬜️ Ignored (Inspect) Visit Preview Dec 1, 2023 3:46pm
sui-kiosk ⬜️ Ignored (Inspect) Visit Preview Dec 1, 2023 3:46pm

@@ -250,7 +250,7 @@
]
},
"32": {
"git_revision": "bdbc732269",
"git_revision": "f918308973-dirty",
Copy link
Contributor

Choose a reason for hiding this comment

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

Technically, this revision should be the new (clean) revision in the release branch, which means you would need to:

  • Put the change out in main for review.
  • Once accepted, create a cherry-pick on the relevant release branch and land it there.
  • Create a snapshot in the release branch.
  • Copy that into the PR on main, and land to main.

But if this works, it works.

Copy link
Contributor

Choose a reason for hiding this comment

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

@amnn We don't need any cherry-picks for this, since the branch cut is scheduled for tomorrow for the v1.15.0 release scheduled to go out on 12/13

@damirka damirka force-pushed the ds/adds-event-to-token branch from b5f8349 to 63c6f52 Compare November 30, 2023 17:49
@vercel vercel bot temporarily deployed to Preview – mysten-ui November 30, 2023 17:50 Inactive
@damirka damirka force-pushed the ds/adds-event-to-token branch from 63c6f52 to 0ac6c81 Compare November 30, 2023 21:34
@vercel vercel bot temporarily deployed to Preview – mysten-ui November 30, 2023 21:35 Inactive
@damirka damirka force-pushed the ds/adds-event-to-token branch from 3f908e4 to bd94770 Compare December 1, 2023 12:14
@vercel vercel bot temporarily deployed to Preview – mysten-ui December 1, 2023 12:15 Inactive
@amnn amnn force-pushed the ds/adds-event-to-token branch from d02fd78 to 83e0312 Compare December 1, 2023 15:45
@vercel vercel bot temporarily deployed to Preview – mysten-ui December 1, 2023 15:46 Inactive
@damirka damirka merged commit 6ec723b into main Dec 1, 2023
34 checks passed
@damirka damirka deleted the ds/adds-event-to-token branch December 1, 2023 16:55
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.

3 participants