coracle/flotilla
1
6
Fork 23
Code Issues 42 Pull Requests 2 Actions Packages Projects 5 Releases Wiki Activity

fix: added logic for password requirements on signup #230

Merged
hodlbod merged 1 commits from DeveshSingh/flotilla:fix/password-requirements into dev 2026-04-17 19:43:27 +00:00
Conversation 0 Commits 1 Files Changed 1
+3
DeveshSingh commented 2026-04-17 19:35:09 +00:00
Contributor
Copy Link
Copy Source

Title

Feat: Show password length requirement on email signup (#224)

Description

The Problem
Previously, when users attempted to sign up via email/password, the 12-character minimum password requirement was a hidden rule. Users only discovered the requirement after submitting the form and triggering an error toast, which caused unnecessary friction during the onboarding flow.

The Solution
This PR improves the UX by explicitly surfacing the password requirement before submission.

  • Modified SignUpEmail.svelte to utilize the info() snippet of the FieldInline component.
  • Added the text "Must be at least 12 characters long." directly underneath the password input field.
  • Formatted the file to adhere to project styling guidelines.

Testing Performed

  • Verified the password requirement text displays correctly below the password input in the "Sign up with Email" modal.
  • Verified standard form submission and client-side validation logic remains unchanged.

Closes #224

### Title Feat: Show password length requirement on email signup (#224) ### Description **The Problem** Previously, when users attempted to sign up via email/password, the 12-character minimum password requirement was a hidden rule. Users only discovered the requirement after submitting the form and triggering an error toast, which caused unnecessary friction during the onboarding flow. **The Solution** This PR improves the UX by explicitly surfacing the password requirement before submission. - Modified `SignUpEmail.svelte` to utilize the `info()` snippet of the `FieldInline` component. - Added the text `"Must be at least 12 characters long."` directly underneath the password input field. - Formatted the file to adhere to project styling guidelines. **Testing Performed** - Verified the password requirement text displays correctly below the password input in the "Sign up with Email" modal. - Verified standard form submission and client-side validation logic remains unchanged. Closes #224
Screenshot 2026-04-18 010439.png
82 KiB
Screenshot 2026-04-18 010439.png
DeveshSingh added 1 commit 2026-04-17 19:35:10 +00:00
fix: added logic for password requirements on signup 0c25327e75
hodlbod merged commit 56edad77a8 into dev 2026-04-17 19:43:27 +00:00
hodlbod deleted branch fix/password-requirements 2026-04-17 19:43:28 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
design
For issues related to design work. Development should be done in another issue and refer back to this one.
 dev
Issues related to development work
 easy
Issues that can be executed on by new contributors
 idea
For issues that aren't yet actionable
 priority
For issues related to high priority features and bugs
 usability
Issues that have to be validated with real live human beings.
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
hodlbod (hodlbod) mplorentz (Matt Lorentz) sakshamjain (Saksham Jain) sondreb triesap userAdityaa (Aditya Chaudhary)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coracle/flotilla#230
Delete Branch "DeveshSingh/flotilla:fix/password-requirements"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?