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

Checkpoint up to 3c1bf4fa5320afe63d344a8e7040c564d99081f8 #964

Closed
wants to merge 2 commits into from

Conversation

paninaro
Copy link
Contributor

@paninaro paninaro commented Aug 25, 2022

Checkpoint up to commit 3c1bf4f

Includes:
Fix tooltip in CAT offer rows #950 (#956)
added fee estimator component to gui (#944)

ronanlong and others added 2 commits August 25, 2022 13:36
* added EstimatedFee component to gui

Signed-off-by: Lipa Long <[email protected]>

* check for fee estimates

Signed-off-by: Lipa Long <[email protected]>

* fee estimator clean-up

Signed-off-by: Lipa Long <[email protected]>

* clean-up

Signed-off-by: Lipa Long <[email protected]>

* more clean-up

Signed-off-by: Lipa Long <[email protected]>

* Remove updated localized sources from the PR

Signed-off-by: Lipa Long <[email protected]>

* Don't use fee estimate controls if RPC returns an error.
Add basic cache control for fee estimate query.

Signed-off-by: Lipa Long <[email protected]>

Signed-off-by: Lipa Long <[email protected]>
Co-authored-by: Jeff Cruikshank <[email protected]>
@paninaro paninaro requested a review from seeden August 25, 2022 21:54
@paninaro paninaro closed this Aug 26, 2022
@paninaro paninaro deleted the paninaro.1_6_0_merge_3c1bf4fa5 branch August 26, 2022 16:36
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