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

Clear cache on native Shapeshift wallet causes the page to load indefinitely #5013

Closed
2 tasks done
MBMaria opened this issue Aug 2, 2023 · 1 comment · Fixed by #5020
Closed
2 tasks done

Clear cache on native Shapeshift wallet causes the page to load indefinitely #5013

MBMaria opened this issue Aug 2, 2023 · 1 comment · Fixed by #5020
Assignees
Labels
needs engineering Requires engineering input before bounty

Comments

@MBMaria
Copy link

MBMaria commented Aug 2, 2023

Overview

When clearing the cache on shapeshift wallet, the page will half reload and then spin continuously. Not seeing this behaviour on MM or KK wallet.

References and additional details

  1. Log into app.shapeshift and connect to shapeshift wallet
  2. In the Settings menu select "Clear Cache"
  3. Note the page starts to refresh but never loads.
  4. To rectify need to disconnect wallet and reconnect.

image

Acceptance Criteria

Clearing cache should result in the page refreshing and being functional

Need By Date

No response

Screenshots/Mockups

No response

Ownership

  • If my bounty needs engineering or needs product I have added the respective labels on the right
  • As the sponsor of this bounty I will review the changes in a preview environment (ops/product) or review the PR (engineering)

Estimated effort

No response

Sponsor / Stakeholder

No response

Bounty Hunters

  • Join our discord
  • Include an expected timeline for you to complete work in the work plan when you apply for this bounty!
  • Please refer to this link for some basic info
  • Please do not start work on this issue until you are approved in Gitcoin.
@MBMaria MBMaria added the needs engineering Requires engineering input before bounty label Aug 2, 2023
@gomesalexandre
Copy link
Contributor

Note this isn't strictly related to cache clear, I have noticed this one on page refresh as well.
Looks like the issue is we're not popping up the wallet connect modal on page refresh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs engineering Requires engineering input before bounty
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants