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

Free TX - free tx every era; governance can update free tx per era #176

Merged
merged 11 commits into from
Sep 7, 2022

Conversation

jakehemmerle
Copy link
Contributor

@jakehemmerle jakehemmerle commented Aug 30, 2022

Feats and updated tests for:

Closes both above tickets. This does NOT include any sybil resistance.

@jakehemmerle jakehemmerle marked this pull request as ready for review August 31, 2022 23:06
JesseAbram
JesseAbram previously approved these changes Sep 1, 2022
Copy link
Member

@JesseAbram JesseAbram left a comment

Choose a reason for hiding this comment

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

maybe making the root call allowed to be changed by council, but that can be a seperate PR

pallets/free-tx/src/lib.rs Outdated Show resolved Hide resolved
@jakehemmerle jakehemmerle merged commit 08df0b3 into master Sep 7, 2022
@jakehemmerle jakehemmerle deleted the jh/free-tx-per-era branch September 7, 2022 17:01
@jakehemmerle jakehemmerle mentioned this pull request Oct 12, 2022
fjarri pushed a commit that referenced this pull request May 5, 2023
)

* feat: free tx per era

governance can set/disable free tx per era
updated mock runtime and with existing tests

* added tests

* updated tests, need to fix era updates in tests

* jesse couldnt even figure this one out!!

* cleanup

* cleanup 2

* removed unneeded comment

* cleaned up tests

* linting

* fixed benchmarks
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.

2 participants