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

created login forms for ssf and pantry #10

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

hellolol2016
Copy link

ℹ️ Issue

Closes

📝 Description

Added a component file for two types of login forms
put them on the root.tsx file to test
added button to switch between the two types of login forms

✔️ Verification

2024-10-2021-34-00-ezgif com-video-to-gif-converter

🏕️ (Optional) Future Work / Notes

missing backend and styling

@hellolol2016 hellolol2016 marked this pull request as ready for review October 21, 2024 01:40
return (
<div>
<Header />
<Outlet />
{ssf ? <SSFLoginForm /> : <PantryLoginForm />}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming this PR corresponds to this ticket, should we be adding this to /landing-page instead? Right now, the login forms show up on every page on the site because all pages show the root content.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

goot catch sam i have moved it to the landing page

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.

3 participants