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

added fee estimator component to gui #944

Merged
merged 7 commits into from
Aug 25, 2022
Merged

added fee estimator component to gui #944

merged 7 commits into from
Aug 25, 2022

Conversation

ronanlong
Copy link
Contributor

Added a component to the gui which displays fee estimations and which takes the fee input from the user

@ronanlong ronanlong requested review from paninaro and seeden August 17, 2022 19:40
paninaro
paninaro previously approved these changes Aug 21, 2022
Copy link
Contributor

@paninaro paninaro left a comment

Choose a reason for hiding this comment

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

approved so that we can continue building off of this.

seeden
seeden previously approved these changes Aug 24, 2022
Copy link
Contributor

@seeden seeden left a comment

Choose a reason for hiding this comment

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

lgtm

ronanlong and others added 7 commits August 25, 2022 12:56
Signed-off-by: Lipa Long <[email protected]>
Signed-off-by: Lipa Long <[email protected]>
Signed-off-by: Lipa Long <[email protected]>
Signed-off-by: Lipa Long <[email protected]>
Add basic cache control for fee estimate query.

Signed-off-by: Lipa Long <[email protected]>
@ronanlong ronanlong dismissed stale reviews from seeden and paninaro via 4b7304b August 25, 2022 19:57
@ronanlong ronanlong force-pushed the ll.fee_estimator-aug8 branch from 0a805b1 to 4b7304b Compare August 25, 2022 19:57
Copy link
Contributor

@paninaro paninaro left a comment

Choose a reason for hiding this comment

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

lgtm

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.

3 participants