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

BTC -> BTC-segwit #92

Merged
merged 3 commits into from
Dec 27, 2023
Merged

BTC -> BTC-segwit #92

merged 3 commits into from
Dec 27, 2023

Conversation

smk762
Copy link
Contributor

@smk762 smk762 commented Dec 25, 2023

Related: KomodoPlatform/coins#905

I believe this fix should solve the advanced orderbook display issues mentioned in chat by decker.

@kivqa kivqa self-requested a review December 27, 2023 11:59
kivqa
kivqa previously approved these changes Dec 27, 2023
Copy link

@kivqa kivqa left a comment

Choose a reason for hiding this comment

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

Looks good for me, I was able to select assets and complete swap on Advanced DEX screen

image

naezith
naezith previously approved these changes Dec 27, 2023
Copy link
Collaborator

@naezith naezith left a comment

Choose a reason for hiding this comment

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

It looks safe, there isn't any code change. It's setting BTC to BTC-segwit for defaultCoins and coinsFiat.

@kivqa
Copy link

kivqa commented Dec 27, 2023

@smk762 smk762 dismissed stale reviews from naezith and kivqa via 7ce53e7 December 27, 2023 12:13
@smk762 smk762 merged commit 36c6afd into dev Dec 27, 2023
1 check passed
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