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.

Sign up for GitHub

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

Jump to bottom

Unused Proxy storage item in Pallet Elections #476

Closed
shawntabrizi opened this issue Mar 16, 2020 · 3 comments
Closed

Unused Proxy storage item in Pallet Elections #476

shawntabrizi opened this issue Mar 16, 2020 · 3 comments
Labels
I10-unconfirmed Issue might be valid, but it's not yet known.

Comments

@shawntabrizi
Copy link
Member

At the moment there is an unused Proxy storage item in pallet-elections and as a result, an "uncallable" extrinsic: proxy_set_approvals.

It seems we are missing a set_proxy and remove_proxy extrinsic.

As the storage item currently exists, it seems that a single account can have multiple proxies set for it (map from proxy => voter :: many => one). So a user should have to pay a deposit for each proxy they set for their account. They can reclaim their deposit by calling remove_proxy.

@gavofyork to clarify if this is the right direction, or if we have other plans.
@shawntabrizi shawntabrizi mentioned this issue Mar 16, 2020
Merged
72 tasks
@kianenigma
Copy link
Contributor

I think we should for now put a warning in the pallet that it is generally not maintained anymore and should not be used.

@ggwpez
Copy link
Member

ggwpez commented Aug 24, 2023

Does this pallet still exist? I dont see pallet-elections.

@juangirini juangirini transferred this issue from paritytech/substrate Aug 24, 2023
@the-right-joyce the-right-joyce added I10-unconfirmed Issue might be valid, but it's not yet known. and removed J2-unconfirmed labels Aug 25, 2023
@ggwpez
Copy link
Member

ggwpez commented Nov 30, 2023

Pallet apparently got deleted. The only Proxy storage item i see is inpallet-recovery.

@ggwpez ggwpez closed this as completed Nov 30, 2023
@github-project-automation github-project-automation bot moved this from 📕 Backlog to ✅ Done in (Nominated) Proof of Stake Nov 30, 2023
helin6 pushed a commit to boolnetwork/polkadot-sdk that referenced this issue Feb 5, 2024
@dependabot
Bump prismjs from 1.24.0 to 1.25.0 in /docs (paritytech#476)
c948ae2 
Bumps [prismjs](https://github.com/PrismJS/prism) from 1.24.0 to 1.25.0.
- [Release notes](https://github.com/PrismJS/prism/releases)
- [Changelog](https://github.com/PrismJS/prism/blob/master/CHANGELOG.md)
- [Commits](PrismJS/prism@v1.24.0...v1.25.0)

---
updated-dependencies:
- dependency-name: prismjs
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
jonathanudd pushed a commit to jonathanudd/polkadot-sdk that referenced this issue Apr 10, 2024
@svyatonik @bkchr
is_finalized_header - use is_finalized flag (paritytech#476)
b0a5e75
This was referenced Jun 5, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
I10-unconfirmed Issue might be valid, but it's not yet known.
Projects
(Nominated) Proof of Stake
Status: ✅ Done
Milestone
No milestone
Development

No branches or pull requests
4 participants
@shawntabrizi @kianenigma @ggwpez @the-right-joyce