Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Pallet staking events to named enum #12342

Merged
merged 12 commits into from
Sep 27, 2022

Conversation

Szegoo
Copy link
Contributor

@Szegoo Szegoo commented Sep 23, 2022

Closes: #12226

Polkadot address: 126X27SbhrV19mBFawys3ovkyBS87SGfYwtwa8J2FjHrtbmA

@Ank4n Ank4n self-requested a review September 23, 2022 22:03
@bkchr bkchr added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Sep 24, 2022
Co-authored-by: Kian Paimani <5588131+kianenigma@users.noreply.github.com>
@kianenigma
Copy link
Contributor

please add your polkadot address to all your PRs @Szegoo :D

@kianenigma
Copy link
Contributor

/tip small

@substrate-tip-bot
Copy link

@kianenigma A small tip was successfully submitted for Szegoo (126X27SbhrV19mBFawys3ovkyBS87SGfYwtwa8J2FjHrtbmA on polkadot).

https://polkadot.js.org/apps/?rpc=wss%3A%2F%2Frpc.polkadot.io#/treasury/tips

@kianenigma
Copy link
Contributor

@Ank4n feel free to merge once you review.

@kianenigma
Copy link
Contributor

bot rebase

@paritytech-processbot
Copy link

Rebased

frame/staking/src/pallet/mod.rs Outdated Show resolved Hide resolved
frame/staking/src/pallet/mod.rs Outdated Show resolved Hide resolved
frame/staking/src/lib.rs Outdated Show resolved Hide resolved
frame/staking/src/pallet/impls.rs Outdated Show resolved Hide resolved
@Ank4n Ank4n added A8-mergeoncegreen and removed A0-please_review Pull request needs code review. labels Sep 27, 2022
@Ank4n
Copy link
Contributor

Ank4n commented Sep 27, 2022

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 2a6c314 into paritytech:master Sep 27, 2022
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* Pallet staking events to named enum

* fmt

* update np staking tests

* update remaining events

* update benchmarks

* Update frame/nomination-pools/test-staking/src/lib.rs

Co-authored-by: Kian Paimani <5588131+kianenigma@users.noreply.github.com>

* Update frame/staking/src/pallet/mod.rs

* Update frame/staking/src/pallet/mod.rs

* Update frame/staking/src/lib.rs

* Update frame/staking/src/pallet/impls.rs

Co-authored-by: Kian Paimani <5588131+kianenigma@users.noreply.github.com>
Co-authored-by: parity-processbot <>
Co-authored-by: Ankan <10196091+Ank4n@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

migrate pallet-staking events to named enum
4 participants