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

Org Root Page: Add Become an Adopter button in hero #994

Open
kasugaijin opened this issue Sep 21, 2024 · 5 comments
Open

Org Root Page: Add Become an Adopter button in hero #994

kasugaijin opened this issue Sep 21, 2024 · 5 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@kasugaijin
Copy link
Collaborator

kasugaijin commented Sep 21, 2024

Please remove these two existing buttons and add a 'Become an Adopter' button (button primary - should be purple) in their place. This link should take the user to the adopter sign up form (same as the sign up button in the top nav).

image
@kasugaijin kasugaijin added good first issue Good for newcomers Ready Make a comment to get assigned. labels Sep 21, 2024
@MarchandMD
Copy link

Do you mean the link_to should point to the new_user_registration_path? (/users/sign_up.... is the "same as the sign up button in the top nav")

What if the user has already signed in?

@kasugaijin
Copy link
Collaborator Author

kasugaijin commented Sep 21, 2024

Haha good point @MarchandMD Yes the new user registration path if not signed in, otherwise it can be another CTA to Browse Pets an go to the adoptable pets path

@MarchandMD
Copy link

Ok cool, thanks for the clarification @kasugaijin

I'll leave this issue open for some other contributor, considering it's a good first issue.

As an unauthenticated user
When I visit /alta/home
I want to see a purple button with the text "Become an Adopter" that points to the new_user_registration_path
So that I'm taken to the sign up path and can become an authenticated user

As an authenticated user
When I visit /alta/home
I want to see a purple button with the text "Browse Pets" that points to the adoptable_pets_path
So that I can easily see the adoptable pets

@noahdurbin
Copy link

I'd like to take care of this, this will be my first issue

@kasugaijin
Copy link
Collaborator Author

@noahdurbin
All yours

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants