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

feat: register button #8

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

vuyaniShabangu
Copy link

adding register button next to login
@pvanheus
Copy link

pvanheus commented May 9, 2023

I'm not getting this code - you added a Register button but what does it do? It seems to also go to INTERNAL_LINKS.Login? Should it not go to NEXT_PUBLIC_KEYCLOAK with some redirect logic as per the existing button here: https://github.com/african-pathogen-archive/portal/blob/develop/components/pages/login/PageContent.tsx#L127 ?

(BTW I think that the existing button - and any new button - should have a configurable redirect that should probably be directed back to the /logged-in page)

@vuyaniShabangu
Copy link
Author

I'm not getting this code - you added a Register button but what does it do? It seems to also go to INTERNAL_LINKS.Login? Should it not go to NEXT_PUBLIC_KEYCLOAK with some redirect logic as per the existing button here: https://github.com/african-pathogen-archive/portal/blob/develop/components/pages/login/PageContent.tsx#L127 ?

(BTW I think that the existing button - and any new button - should have a configurable redirect that should probably be directed back to the /logged-in page)

I think the conversation around what we're still going to do with the login button was still ongoing.
So currently the login button takes you the the /login route, where a user can log in or request an account.
So I figured register should also go there since they can follow the request an account flow.

Or should I link to the specific key cloak pages?

@pvanheus
Copy link

I'm not getting this code - you added a Register button but what does it do? It seems to also go to INTERNAL_LINKS.Login? Should it not go to NEXT_PUBLIC_KEYCLOAK with some redirect logic as per the existing button here: https://github.com/african-pathogen-archive/portal/blob/develop/components/pages/login/PageContent.tsx#L127 ?
(BTW I think that the existing button - and any new button - should have a configurable redirect that should probably be directed back to the /logged-in page)

I think the conversation around what we're still going to do with the login button was still ongoing. So currently the login button takes you the the /login route, where a user can log in or request an account. So I figured register should also go there since they can follow the request an account flow.

Or should I link to the specific key cloak pages?

I think either we need a Login / Register button (perhaps this is preferable) or the Register button should go to a (configurable - but in our case keycloak) URL.

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

Successfully merging this pull request may close these issues.

2 participants