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

feat: support conditional payment for submission and make them operational #71

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mattsse
Copy link
Contributor

@mattsse mattsse commented Jun 2, 2021

Changes

The submit is marked as operational with DispatchClass

Whether operators should be pay the fee for submitting values (submit) can now be configured with SubmitterPaysFee

clearloop and others added 4 commits June 2, 2021 15:07
* feat(feed): make submit extrinsic in feed-pallet operational

* feat(feed): custom weight for call submit

* feat(feed): add parameter oracle for weight calculation

* feat(feed): make SubmitPaysFee configurable

* fix(feed): SubmitPaysFee config in runtime

* perf(feed): rename to SubmitterPaysFee

* perf(feed): free for valid round

* perf(feed): move weight calculation inside extrinsic

* perf(feed): remove useless .clone() s

* perf(feed): clean imports in default_weight.rs

* perf(node): ensure benchmarks works

* perf(feed): move paysfee checking to the end of function submit

* feat(feed): checkout the checking of ensure_valid_round

* perf(feed): fix the tests of feed
* feat(feed): weight function for set_pruning_window

* feat(feed): function count_overflow_winodws for determining weights

* fix(feed): fix becnhmarks according to previous updates

* feat(feed): benchmark set_pruning_window with param

* chore(feed): updates the weight of set_pruning_window

* chore(feed): remove function count_overflow

* feat(feed): re-generate weights for pallet_chainlink_feed, 20210520

* fix(node): fix the weight configuration in node-template and update default_weights in feed pallet
Copy link
Contributor

@ebarakos ebarakos left a comment

Choose a reason for hiding this comment

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

Can we add a simple test case as well @mattsse ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants