Skip to content

feat: add fullstack capstone project demo - #700

Open
Ksound22 wants to merge 6 commits into
freeCodeCamp:mainfrom
Ksound22:feat/add-fullstack-capstone-project-demo
Open

Ksound22 wants to merge 6 commits into
freeCodeCamp:mainfrom
Ksound22:feat/add-fullstack-capstone-project-demo

Conversation

@Ksound22

@Ksound22 Ksound22 commented Sep 9, 2026

Copy link
Copy Markdown
Member

Checklist:

Closes #XXXXX

@Ksound22
Ksound22 requested a review from a team as a code owner September 9, 2026 13:55
@socket-security

This comment was marked as resolved.

@jdwilkin4 jdwilkin4 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Just started testing this locally and noticed there wasn't a place to toggle a show/hide password for the signup page.

Screenshot 2026-09-14 at 8 09 07 PM

I think that should be added as a nice UX improvement and is a common pattern for forms.

@jdwilkin4 jdwilkin4 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

To me this Invalid request data. banner at the top when you fill out the form incorrectly isn't needed.

Image

You are already showing validation error messages next to the fields which should be enough.

Image

@jdwilkin4 jdwilkin4 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The second Profile text here isn't needed.

Image

@jdwilkin4 jdwilkin4 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I created an org and noticed this toggle at the top.
Not sure what it does but it looks like a bug to me.

toggle-collaborer.mov

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