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

Carolyn/signup page #81

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

Carolyn/signup page #81

wants to merge 11 commits into from

Conversation

carolynzhang18
Copy link
Collaborator

@carolynzhang18 carolynzhang18 commented Nov 21, 2024

Notion ticket link

Welcome/Login/Signup Page Design Implementation

Implementation description

  • Implemented designs for the welcome page (/welcome)
  • Implemented designs for side drawers (no error handling)
    • Admin login
      • Info text after clicking "Don't Have and Account?"
    • Facilitator login
  • Implemented designs for sign up components (/signup) (no error handling)

Figma Design:
https://www.figma.com/design/BVjAeCcvNSYMyaf9Vd9T6M/Early-Stages-(S24)?node-id=299-5521&t=5tCNi804SCHj8iJd-1

Go to --- DEV HANDOFF --- > Login

Steps to test

Test case 1: Login functionality still works

Test case 2: Facilitator sign up functionality still works

Test case 3: All links/url directs to proper page

Below are the implemented links:

  • Facilitator sign up from /welcome page directs to /signUp page
  • Facilitator sign up from facilitator login side drawer directs to /signUp page
  • Clicking "Already Have an Account" from /signUp page redirects to /welcome page with facilitator login side drawer open
  • Clicking any "Forgot Password" text leads to /forgot-password` page

What should reviewers focus on?

  • Design is accurately implemented
  • Implementation does not break with various screen sizes

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@carolynzhang18 carolynzhang18 mentioned this pull request Nov 21, 2024
4 tasks
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