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

Re-consideration poke for Deposits #5591

Open
kianenigma opened this issue Sep 4, 2024 · 3 comments
Open

Re-consideration poke for Deposits #5591

kianenigma opened this issue Sep 4, 2024 · 3 comments
Assignees
Labels
T1-FRAME This PR/Issue is related to core FRAME, the framework. T2-pallets This PR/Issue is related to a particular pallet.

Comments

@kianenigma
Copy link
Contributor

Many pallets are collecting deposits to prevent spam. Some hard-code values that indicate what that deposit is, some use the newer Consideration API. Some efforts exist to migrate more pallets to the new API: #226

In both cases, the pallets should be capable of adjusting the deposit in a lazy manner. As in, if the "deposit formula" (regardless of how it is expressed) changes, we can re-adjust it.

This should be done lazily, via a permissionless fn poke() transaction that is free in the case of a successful adjustment, and a paid noop otherwise.

@kianenigma kianenigma added T1-FRAME This PR/Issue is related to core FRAME, the framework. T2-pallets This PR/Issue is related to a particular pallet. labels Sep 4, 2024
@xlc
Copy link
Contributor

xlc commented Sep 4, 2024

Yeah we should have lot more utility pallets to reduce number of non-business logic code in each pallet.
pallet-parameters is one improve that eliminates the needs to implement parameter handling logics. We could have a similar pallet handles all the deposits and poke etc.

@kianenigma
Copy link
Contributor Author

good name: fn reconisder(origin).

@kianenigma
Copy link
Contributor Author

I would like to see one example of this created as a learning demo, and it would allow then many ppl to start working on this in parallel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T1-FRAME This PR/Issue is related to core FRAME, the framework. T2-pallets This PR/Issue is related to a particular pallet.
Projects
None yet
Development

No branches or pull requests

3 participants