Skip to content

Feat/implement dynamic fee rate based on fee rate statistics - #2587

Closed
alexsupa597 wants to merge 11 commits into
nervosnetwork:developfrom
alexsupa597:feat/implement-dynamic-fee-rate-based-on-fee-rate-statistics
Closed

Feat/implement dynamic fee rate based on fee rate statistics#2587
alexsupa597 wants to merge 11 commits into
nervosnetwork:developfrom
alexsupa597:feat/implement-dynamic-fee-rate-based-on-fee-rate-statistics

Conversation

@alexsupa597

@alexsupa597 alexsupa597 commented Feb 13, 2023

Copy link
Copy Markdown

What problem does this PR solve?

As title.

An RPC to fetch fee_rate_statistics is added since CKB@v0.106.0, which returns mean fee rate and median fee rate of the specified block range.

With the historic data, Neuron could offer more precise fee rate recommendations for users.

PRD: Magickbase/neuron-public-issues#87
UI: https://www.figma.com/file/6XNoimRDbFTTNm016rbIdU/Magickbase?node-id=319%3A4539&t=oQrWTpg2GhW4obRJ-0

Ref: Magickbase/neuron-public-issues#95

Check List

Test

e2e Test

Task

none

Keith-CY and others added 9 commits March 1, 2023 01:11
MacOS is specified by version number explicitly because tag
"macos-latest" refers to macos-11 or macos-12 randomly during
GitHub action's transition while macos 12 removed Python 2 which
is required by electron-builder.

Since latest electron-builder has supported python3, this hotfix
will not introduce python2 in host but downgrade it to macos 12

Ref: electron-userland/electron-builder#6606
@Keith-CY

Keith-CY commented Mar 3, 2023

Copy link
Copy Markdown
Contributor

Most commits in this PR are not signed

@alexsupa597

Copy link
Copy Markdown
Author

Most commits in this PR are not signed

Yes, it is. I'll create another PR to fix this problem. It maybe cause I use git push --force to push the code.

@alexsupa597

alexsupa597 commented Mar 4, 2023

Copy link
Copy Markdown
Author

@Keith-CY The new PR is over here. Please help to make a code review. Thx.
And I'll close this PR.
#2599

@alexsupa597 alexsupa597 closed this Mar 4, 2023
@alexsupa597
alexsupa597 deleted the feat/implement-dynamic-fee-rate-based-on-fee-rate-statistics branch March 28, 2023 15:57
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