-
Notifications
You must be signed in to change notification settings - Fork 104
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
Comments
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? |
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 |
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 As an authenticated user |
I'd like to take care of this, this will be my first issue |
@noahdurbin |
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).
The text was updated successfully, but these errors were encountered: