search

hlxsites / zemax

AEM Franklin Solution for https://www.zemax.com/
https://main--zemax--hlxsites.hlx.page/
Apache License 2.0
3 stars 1 forks source link

Login Redirect #220

Closed Anthony-Hopkins-star closed 1 year ago

Anthony-Hopkins-star commented 1 year ago

After logging in, the user should be redirected back to the profile page.

kunwarsaluja commented 1 year ago

@synox with your latest changes, should this have been fixed already ?

synox commented 1 year ago

yes

kunwarsaluja commented 1 year ago

@synox I don't see the redirect happening to the /pages/profile after login in the main branch like it's happening in the zemax live site(zemax.com).

synox commented 1 year ago

@kunwarsaluja Ah, I only fixed it for the case where you initiate it from https://main--zemax--hlxsites.hlx.page/pages/profile.

Changing https://github.com/hlxsites/zemax/blob/25899093da400365bb82abf9e7edaa2245e95a92/blocks/header/header.js#L270-L274 to always return the profile url should fix it. @kunwarsaluja can you try this?