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.

Sign up for GitHub

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

Jump to bottom

Trading - deduplicate account in redux #16524

Closed
tomasklim opened this issue Jan 22, 2025 · 1 comment · Fixed by #16681
Closed

Trading - deduplicate account in redux #16524

tomasklim opened this issue Jan 22, 2025 · 1 comment · Fixed by #16681
Assignees
@FreeWall @adderpositive
Labels
code Code improvements +Invity Related to Invity project

Comments

@tomasklim
Copy link
Member

Please deduplicate account used for trading in redux to avoid potentional future bugs when account is outdated.

#16442 (review)
@tomasklim tomasklim added the code Code improvements label Jan 22, 2025
@github-project-automation github-project-automation bot moved this to 🎯 To do in Suite Desktop Jan 22, 2025
@tomasklim tomasklim added the +Invity Related to Invity project label Jan 22, 2025
@tomasklim
Copy link
Member Author

Image

@tomasklim tomasklim mentioned this issue Jan 23, 2025
Merged
@adderpositive adderpositive mentioned this issue Jan 28, 2025
Merged
@adderpositive adderpositive linked a pull request Jan 28, 2025 that will close this issue
Refactor/trading account state #16681
Merged
@github-project-automation github-project-automation bot moved this from 🎯 To do to 🤝 Needs QA in Suite Desktop Feb 3, 2025
@bosomt bosomt moved this from 🤝 Needs QA to ✅ Approved in Suite Desktop Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@FreeWall FreeWall

@adderpositive adderpositive

Labels
code Code improvements +Invity Related to Invity project
Projects
Suite Desktop
Status: ✅ Approved
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactor/trading account state trezor/trezor-suite
3 participants
@FreeWall @adderpositive @tomasklim