Skip to content

Conversation

@jyy009
Copy link

@jyy009 jyy009 commented May 26, 2024

Netlify link

https://heroic-beignet-77e068.netlify.app

Render

https://project-auth-kh49.onrender.com/

Collaborators

[viktoria-olo, schouenkes]

AntonellaMorittu and others added 30 commits May 22, 2024 14:57
Co-authored-by: Maja Schouenke <schouenkes@users.noreply.github.com>
Co-authored-by: Maja Schouenke <schouenkes@users.noreply.github.com>
Co-authored-by: viktoria-olo <viktoria-olo@users.noreply.github.com>
Co-authored-by: jyy009 <jyy009@users.noreply.github.com>
Co-authored-by: jyy009 <jyy009@users.noreply.github.com>
split up formData state management
add expressListEndpoints and update readme
Copy link
Contributor

@AntonellaMorittu AntonellaMorittu left a comment

Choose a reason for hiding this comment

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

Hello girls, good job implementing the auth for your final project! The app works well and it meets the requirements. I recommend to validate the input values like postcode and city, and add some rules to password or username (at the moment you can input letters in the postcode, numbers for city, and psw and username be just 1 character). it could be done either in the BE, for example:

password: { 
    type: String, 
    required: true,
    validate: {
        validator: function(v) {
            return v.length >= 8;
        },
        message: props => `Password should be at least 8 characters long!`
    }
}

or in the FE. Other than that you're on a great track so keep it up! 💯

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.

4 participants