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

hashPassword user argument is in PascalCase #205

Closed
Melchyore opened this issue Aug 20, 2022 · 0 comments · Fixed by #206
Closed

hashPassword user argument is in PascalCase #205

Melchyore opened this issue Aug 20, 2022 · 0 comments · Fixed by #206
Labels
Good First Issue Want to contribute? Just filter by this label Priority: Low Something worth considering, but not a top priority for the team Type: Bug The issue has indentified a bug

Comments

@Melchyore
Copy link

Package version

8.2.2

Node.js and npm version

16.17.0

Sample Code (to reproduce the issue)

When you configure the package, it will generate a User model. After the last release, the argument user of the hashPassword method is in PascalCase, so we'ell end up with class name === argument name.

BONUS (a sample repo to reproduce the issue)

@RomainLanz RomainLanz added Type: Bug The issue has indentified a bug Priority: Low Something worth considering, but not a top priority for the team Good First Issue Want to contribute? Just filter by this label labels Aug 23, 2022
RomainLanz added a commit that referenced this issue Aug 23, 2022
thetutlage pushed a commit that referenced this issue Aug 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue Want to contribute? Just filter by this label Priority: Low Something worth considering, but not a top priority for the team Type: Bug The issue has indentified a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants