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

Fix trade confirm modal ticker icons wrong #2134

Merged
4 commits merged into from
Dec 6, 2022
Merged

Conversation

ghost
Copy link

@ghost ghost commented Nov 30, 2022

Might fix #2105, hard to replicate, waiting for more feedbacks!

@ghost ghost requested review from smk762, SirSevenG and Canialon November 30, 2022 15:08
@tonymorony
Copy link

accordingly to the reports the problem is not just in icon but actually wrong tickers passing (#2105 (comment) there is an marketmaker error that Base/Rel tickers can't be the same in this case)

Was you able to reproduce the issue on your side @SylEze ?

@ghost
Copy link
Author

ghost commented Nov 30, 2022

accordingly to the reports the problem is not just in icon but actually wrong tickers passing (#2105 (comment) there is an marketmaker error that Base/Rel tickers can't be the same in this case)

Was you able to reproduce the issue on your side @SylEze ?

The only issue was the icons on my side
If you look at the text below, the tickers displayed are the good ones and I was able to make a swap when the icons were wrong

@smk762
Copy link
Collaborator

smk762 commented Nov 30, 2022

conflicts need to be resolved ^

@smk762 smk762 assigned ghost Dec 2, 2022
Copy link
Collaborator

@smk762 smk762 left a comment

Choose a reason for hiding this comment

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

I have been unable to replicate the initial issue

Copy link

@SirSevenG SirSevenG left a comment

Choose a reason for hiding this comment

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

Was not able to reproduce intial issue

@ghost ghost merged commit 5a8d779 into dev Dec 6, 2022
@smk762 smk762 deleted the fix/trade-conf-modal-ticker-same branch August 7, 2023 07:42
This pull request was closed.
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.

[BUG] Trade confirmation tickers not same as selected in order
3 participants