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

Reconnect to Metamask after page refresh. #28

Open
4 of 6 tasks
dele22 opened this issue Jun 23, 2021 · 1 comment
Open
4 of 6 tasks

Reconnect to Metamask after page refresh. #28

dele22 opened this issue Jun 23, 2021 · 1 comment

Comments

@dele22
Copy link

dele22 commented Jun 23, 2021

Describe the bug
Every time I refresh a page a "Connect to a wallet" page appears regardless of whether I have connected before or not.

To Reproduce
Steps to reproduce the behaviour:

  1. Go to 'https://alpha.pstake.finance/'
  2. Connect to your Metamask wallet
  3. Click 'refresh page' or press F5
  4. See error

Expected behaviour
I would expect (like other DeFi apps) that if you are already connected to your wallet then you dont have to again.

Screenshots
Capture

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser: Brave
  • Version: 1.26.67

Ethereum address
0x083bB8D8732A98Efd4F45F41DEDb08f6f67c29a3

Criticality Assessment
Please pick one:

  • High: An issue that might cause immediate loss of the funds or permanent/severe damage of the protocol state
  • Medium: An issue that might theoretically cause minimal loss of funds damage the protocol state or cause severe user dissatisfaction
  • Low: An issue that might cause user dissatisfaction or minimal failure of the application

Checklist

  • The reported issue is in the scope of the pStake BugBounty program.
  • This issue has not been reported before.
  • The ethereum address filled in is valid.
@kombos
Copy link

kombos commented Jul 27, 2021

this is a single page app which follows the onLoad-onRefresh design methodology for connecting and disconnecting wallet. This is not a bug but the way app is designed, not different from banking and financial apps which logs out on refresh. you are not supposed to use refresh button at any point of time in the app, unless expressly instructed in the app itself.

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

No branches or pull requests

2 participants