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

[QA] Refreshing while in the User Account area redirects you to the landing page #4206

Open
melyndav opened this issue Jan 29, 2025 · 8 comments · May be fixed by #2448
Open

[QA] Refreshing while in the User Account area redirects you to the landing page #4206

melyndav opened this issue Jan 29, 2025 · 8 comments · May be fixed by #2448
Assignees
Labels
bug Something isn't working Priority: P2 (Medium)

Comments

@melyndav
Copy link

melyndav commented Jan 29, 2025

Steps to reproduce

  1. In the QA env, navigate to your user account area.
  2. Reload the page.
  3. You will be redirected to the landing page and taken away from the user account area.

Expected behaviour

  • Refreshing or hard refreshing any page while in the user account area should ensure you remain in the user account area.

This is not happening in production, and it would be nice for this not to be introduced into production given there are circumstances like avatar uploads and KYC content that users may be inclined to refresh in order to see up-to-date images/ information/ payment history/ payment information etc.

Actual behaviour

  • While in the user account area, refreshing redirects you unexpectedly to the landing page.

I was using an embedded wallet.

@melyndav melyndav added bug Something isn't working Priority: P2 (Medium) labels Jan 29, 2025
@melyndav melyndav changed the title QA - Refreshing while in the User Account area redirects you to the landing page [QA] Refreshing while in the User Account area redirects you to the landing page Jan 29, 2025
@rdig
Copy link
Member

rdig commented Jan 29, 2025

I know what this is happening, but since we no longer control the autologin functionality of the wallet I can't really do all that much

I'll investigate what our options are thought, but might not be possible in the end

@rdig rdig self-assigned this Jan 29, 2025
@melyndav
Copy link
Author

I was just redirected while trying to finalize a motion. I am trying to replicate this, but it's a bit concerning, I also experienced this inside the context of a colony and in an active motion.

@melyndav
Copy link
Author

Update on this one @rdig - it's still happening with the embedded wallet from the user account. I am currently trying to test the redirection from inside the colony when trying to finalize a motion I experienced yesterday.

@melyndav
Copy link
Author

@rdig another update (from Discord)

I was able to replicate the redirection issue just now, however while being connected to Metamask.

I was creating a new action (Mint tokens) using permissions, selected the CTA and was immediately redirected to the landing page, in a disconnected state (screenshot of where I landed).

Image

I experienced this yesterday after selecting the CTA in the 'Finalize' widget of an Advanced Payment.

I will continue testing this as priority and catch a screen recording for clarity.

@rdig
Copy link
Member

rdig commented Jan 30, 2025

The last comment is totally different than what this initial issue is describing.

I will create a new issue to track this

@rdig
Copy link
Member

rdig commented Jan 30, 2025

@rdig another update (from Discord)

I was able to replicate the redirection issue just now, however while being connected to Metamask.

I was creating a new action (Mint tokens) using permissions, selected the CTA and was immediately redirected to the landing page, in a disconnected state (screenshot of where I landed).

Image

I experienced this yesterday after selecting the CTA in the 'Finalize' widget of an Advanced Payment.

I will continue testing this as priority and catch a screen recording for clarity.

This is now being tracked in JoinColony/colonyCDappAuthProxy#35

@melyndav
Copy link
Author

melyndav commented Feb 7, 2025

Also this one is well @rdig @jakubcolony?

@rdig
Copy link
Member

rdig commented Feb 7, 2025

This will be automatically closed by my #2448 which actually fixed this issue, but since that wasn't merged into master yet, this is still open

Once I merge it, it will automatically close this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Priority: P2 (Medium)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants