Skip to content

MVP of Auth0 signup flow, additional signup form, dashboard access (styling per Figma design). #8

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

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

Conversation

stevenseb
Copy link

@stevenseb stevenseb commented May 5, 2025

Attached to issue: #6

Still much to do on finer details:

  1. Likely need to get the Figma logo version updated in the code
  2. May need to get with MK to review application of color palette to make sure it is all correct
  3. Attempted to solve known issue with Auth0 and SPA given that hitting back button after registration can result in Auth0 error page since the connection is closed once registration complete (attempted a fix in /components/BackNavigationFix.tsx but doesn't seem to work
  4. Still need to render login button, component with logic in /auth0 folder however, need to discuss entire header layout, proper logged-in button views etc. so we can get it all correct together
  5. Used '/' root route to display a mock-up landing page to simulate hitting the sign-up button from WP site or external source to test the entire flow and created a PublicHeader component based off of the Header component but we have 2 headers with slightly different styling, but really the PublicHeader.tsx is just for testing/dev so, need to focus on making Header.tsx work as it should for logged in user.
  6. Have not really implemented or thoroughly tested mobile styling, just have some basic minimums.
  7. Need to resolve the consent popup which we don't want right now.
  8. May need to enhance the form validator as it's only checking completion right now.
  9. Still plan to refactor some code to separate concerns and improve readability.
  10. Probably will be helpful if we check global styles and set more of them as appropriate.
  11. I moved any css stylesheets into /styles folder inside of /components and /pages and a mockDB file is inside the /utils folder.
  12. Just a fairly good MVP right now to see if on the right track

Welcome feedback on styling and thorough testing to find any other potential bugs. Next steps are verify/correct styling per MK and Figma designs and fully code the logic for header and buttons when logged in. Functionality requires .env variables which I can share with Peter via Slack if that's acceptable.

image

{874482EF-97A0-4534-BE55-2AB147C74CEC}

{05AF4CA5-C5CB-42A2-ADDF-F21895EC649D}

localhost_5173_signup

Copy link

netlify bot commented May 5, 2025

Deploy Preview for helpful-platform ready!

Name Link
🔨 Latest commit
🔍 Latest deploy log https://app.netlify.com/projects/helpful-platform/deploys/6828ee266e7c9c563274594b
😎 Deploy Preview https://deploy-preview-8--helpful-platform.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Jbutler-helpful
Copy link

Jbutler-helpful commented May 13, 2025

Yes!!! it deployed, I can't get to the top registration page using the sign in/sign up

@mkharkar
Copy link

mkharkar commented May 15, 2025

Hey @stevenseb ! Thanks for sharing the screenshots. Few quick notes -

  1. The styling of the field boxes needs to be updated—please refer to the final version here: https://www.figma.com/design/Q2OzxOL3BB6aqwuXBGZG0w/Low-Fidelity?node-id=68-2963&t=ZL2zjetwQwHS6MNq-1 for reference.
  2. The spacing between fields could use a bit more breathing room and should be consistent throughout.
  3. For the selected skill pills, the grey background color from the final design needs to be applied here as well.
  4. The top nav bar should follow the light grey style that we’ve used consistently across other pages.
  5. Overall, we just need to tighten up spacing, color usage, and box styles to align with the final design.

@psprings psprings added the enhancement New feature or request label May 15, 2025
@psprings psprings linked an issue May 15, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

We need a signup + flow that includes basic authentication
4 participants