Skip to content
This repository was archived by the owner on Jun 24, 2022. It is now read-only.

[Recent Orders] - Orders displayed in the wrong network #1785

Closed
MareenG opened this issue Nov 5, 2021 · 7 comments
Closed

[Recent Orders] - Orders displayed in the wrong network #1785

MareenG opened this issue Nov 5, 2021 · 7 comments
Assignees
Labels
app:CowSwap CowSwap app Bug Something isn't working Medium Severity indicator. It causes some undesirable behavior, but the system is still functional

Comments

@MareenG
Copy link

MareenG commented Nov 5, 2021

Description
First reported in PR #1639
#1639 (comment)
#1639 (comment)
Also appeared in #1783

Some orders of other network are displayed in the activities overview.
Eg. Xdai trades in Mainnet or Rinkeby trades when selecting xDai

First it only happened when using FF or Brave now also for Chrome

How to Reproduce
Steps to reproduce the behavior:

  1. Go to PRs 1639 or 1783 and open cowswap
  2. Perform trades in different networks
  3. Open trading activites and cross check the trades
  4. trades are displayed in wrong network

Expected behavior
xDai trades only shown when xDai selected
Mainnet for Mainnet
Rinkeby for Rinkeby

Screenshots

Bildschirmfoto 2021-11-05 um 10 35 13

Bildschirmfoto 2021-11-05 um 11 03 56

Bildschirmfoto 2021-11-05 um 10 33 53

Bildschirmfoto 2021-11-05 um 10 33 28

@MareenG MareenG added app:CowSwap CowSwap app Medium Severity indicator. It causes some undesirable behavior, but the system is still functional Bug Something isn't working labels Nov 5, 2021
@MareenG
Copy link
Author

MareenG commented Nov 5, 2021

Also it looks like it doesn't properly update the recent activities of the wrong network:
Bildschirmfoto 2021-11-05 um 11 40 03

@MareenG
Copy link
Author

MareenG commented Nov 5, 2021

Deleting redux_localstorage_simple_orders cleared the order activites and the orders of the wrong network where removed

@elena-zh
Copy link

elena-zh commented Nov 5, 2021

It seems that I faced a similar issue a couple of weeks ago #1618
Anyways, let these issues be linked.

@alfetopito
Copy link
Contributor

To be clear, is this happening ONLY on my PR where orders are loaded from api or ALSO regular loading (stored in the browser)?

@MareenG
Copy link
Author

MareenG commented Nov 8, 2021

@alfetopito The orders displayed in the wrong network ONLY happen in your PR (#1639) and the one from Anxo (#1783)

That orders might not update properly happened to Elena before as described in #1618

@alfetopito
Copy link
Contributor

Hmm, then this is more general and not necessarily related to account separation nor orders from API, as reported by Elena on #1618
At least we know this hasn't been introduced with any of those changes.

I suspect this might be there for awhile and it should be reproducible on develop / 1.5 branches, if introduced with the SC integration, or even be on prod.

Need more debugging to find out what's causing it

@elena-zh
Copy link

elena-zh commented Dec 3, 2021

No longer reproducible.

@elena-zh elena-zh closed this as completed Dec 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
app:CowSwap CowSwap app Bug Something isn't working Medium Severity indicator. It causes some undesirable behavior, but the system is still functional
Projects
None yet
Development

No branches or pull requests

3 participants