-
Notifications
You must be signed in to change notification settings - Fork 643
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
include events for voting #4613
include events for voting #4613
Conversation
…tion-voting' into dami/add-events-to-pallet-conviction-voting
Any possibility of help getting this reviewed @muharem |
@@ -170,6 +170,10 @@ pub mod pallet { | |||
Delegated(T::AccountId, T::AccountId), | |||
/// An \[account\] has cancelled a previous delegation operation. | |||
Undelegated(T::AccountId), | |||
/// An account that has voted | |||
VoteEnacted { who: T::AccountId, vote: AccountVote<BalanceOf<T, I>> }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
VoteEnacted { who: T::AccountId, vote: AccountVote<BalanceOf<T, I>> }, | |
Voted { who: T::AccountId, vote: AccountVote<BalanceOf<T, I>> }, |
Enacted sounds to me more final.
prdoc/pr_4613.prdoc
Outdated
@@ -0,0 +1,10 @@ | |||
title: "Include events for vote and remove vote" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
title: "Include events for vote and remove vote" | |
title: "pallet-conviction-voting: Include events for vote and remove vote" |
Otherwise the CHANGELOG will read a bit weird.
@dharjeezy please address the review comments, and we can get this merged |
…tion-voting' into dami/add-events-to-pallet-conviction-voting
done |
prdoc/pr_4613.prdoc
Outdated
doc: | ||
- audience: Runtime User | ||
description: | | ||
Includes event for vote and remove_vote |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd mention what the fields are
Co-authored-by: Muharem <ismailov.m.h@gmail.com>
looks good, I merge when CI pass |
2982bf3
closes paritytech#143 Polkadot address: 12GyGD3QhT4i2JJpNzvMf96sxxBLWymz4RdGCxRH5Rj5agKW --------- Co-authored-by: Muharem <ismailov.m.h@gmail.com>
* master: (27 commits) Bridges improved tests and nits (#5128) Fix misleading comment about RewardHandler in epm config (#3095) Introduce a workflow updating the wishlist leaderboards (#5085) membership: Restructure pallet into separate files (#4536) Fix after ring-proof api change (#5126) Bump paritytech/review-bot from 2.4.0 to 2.5.0 (#5057) Bump docker/login-action from 3.0.0 to 3.3.0 (#5109) Bump docker/build-push-action from 5.1.0 to 6.5.0 (#5108) Bump peter-evans/create-pull-request from 5.0.0 to 6.1.0 (#5093) Tx Payment: drop ED requirements for tx payments with exchangeable asset (#4488) Remove `pallet-getter` usage from pallet-transaction-payment (#4970) pallet macro: do not generate try-runtime related code when frame-support doesn't have try-runtime. (#5099) fix(chain-spec): ChainSpecBuilder with object as default genesis (#4345) Migrate BEEFY BLS crypto to bls12-381 curve (#4931) Bump clap from 4.5.9 to 4.5.10 in the known_good_semver group (#5120) Use jobserver in wasm-builder to limit concurrency of spawned cargo processes (#4946) include events for voting (#4613) [subsystem-bench] Add mocks for own assignments triggering (#5042) Remove not-audited warning (#5114) hotfix: blockchain/backend: Skip genesis leaf to unblock syncing (#5103) ...
closes #143
Polkadot address: 12GyGD3QhT4i2JJpNzvMf96sxxBLWymz4RdGCxRH5Rj5agKW