-
-
Notifications
You must be signed in to change notification settings - Fork 139
@accounts/mongo : Monorepo integration #176
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
Conversation
* Setup
Create and find user
Add sessions support
Don't throw on setProfile
Fix missing profile to user
bugfix(object-id): added conversion to ObjectID when needed
Update README
Fix user.id not set
Mongoose doesn’t set that
Add functions for email verification and password change
Add reason param to addResetPasswordToken
added umd build
Added a flag that allows for user collection to have simple string ids
Implemented setResetPassword method
case insensitive user name query
changes for token based sessions
Add unsetService function
Codecov Report
@@ Coverage Diff @@
## master #176 +/- ##
==========================================
+ Coverage 92.16% 93.19% +1.03%
==========================================
Files 42 44 +2
Lines 1034 1191 +157
Branches 124 167 +43
==========================================
+ Hits 953 1110 +157
Misses 70 70
Partials 11 11
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
packages/database-mongo/tslint.json
Outdated
@@ -0,0 +1,5 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can remove this file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops done :)
packages/database-mongo/package.json
Outdated
"coverage": "yarn testonly --coverage", | ||
"prepublishOnly": "yarn compile" | ||
}, | ||
"renovate": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you remove renovate config also please?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also done, along with the circleci folder and the .gitignore
…ved tslint.json + removed .circleci folder
Good to merge @Aetherall |
No description provided.