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

WIP: Transaction Fee Multiplier#2847

Closed
peektism wants to merge 1 commit into
masterfrom
amar/fee-multiplier
Closed

WIP: Transaction Fee Multiplier#2847
peektism wants to merge 1 commit into
masterfrom
amar/fee-multiplier

Conversation

@peektism
Copy link
Copy Markdown
Contributor

Implementation of the slow fee mechanism described in detail here.

This addresses #2430 and builds on top of #2799

@parity-cla-bot
Copy link
Copy Markdown

It looks like @4meta5 hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement at the following URL: https://cla.parity.io

Once you've signed, please reply to this thread with [clabot:check] to prove it.

Many thanks,

Parity Technologies CLA Bot

1 similar comment
@parity-cla-bot
Copy link
Copy Markdown

It looks like @4meta5 hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement at the following URL: https://cla.parity.io

Once you've signed, please reply to this thread with [clabot:check] to prove it.

Many thanks,

Parity Technologies CLA Bot

@peektism
Copy link
Copy Markdown
Contributor Author

[clabot:check]

@parity-cla-bot
Copy link
Copy Markdown

It looks like @4meta5 signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

1 similar comment
@parity-cla-bot
Copy link
Copy Markdown

It looks like @4meta5 signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

Comment thread srml/system/src/lib.rs
/// Total length in bytes for all extrinsics put together, for the current block.
AllExtrinsicsLen: Option<u32>;
/// Total weight for all extrinsics put together, for the current block.
AllExtrinsicsWeight: Option<u32>;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is already included in the old PR. it is more or less stable, so you should/could merge it in here now.

Comment thread srml/system/src/lib.rs
pub fn all_extrinsics_len() -> u32 {
<AllExtrinsicsLen<T>>::get().unwrap_or_default()
/// Gets a total weight of all executed extrinsics.
pub fn all_extrinsics_weight() -> u32 {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same; already exists.

@peektism peektism closed this Jun 12, 2019
@peektism peektism deleted the amar/fee-multiplier branch June 12, 2019 12:48
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.

4 participants