Uh oh!
There was an error while loading. Please reload this page.
Update/borrow - #261
Merged
Merged
Conversation
* test: add integration tests * fix: some stuff and add an assertion
* test: add integration tests fix: time * fix and add stuffs * cleanup
* add: reset password token entity, configuration, and added to app db context also renamed the parameter of OnModelCreating in ApplicationDbContext.cs to modelBuilder to satisfy SonarLint * chore: refactored SendResetPasswordHtmlMail method of IMailService.cs also: - Pass a reset password token hash so frontend can call the right link - Change one template variable name to ResetPasswordTokenHash and send the token along - Renamed HTMLMailData.cs to HtmlMailData.cs to satisfy sonarlint * add: IAuthDbContext.cs and registered it * chore: refactored IdentityService to be using abstractions instead of implementations * chore: add reset token creation logic to UserCreatedEventHandler.cs * chore: UserId foreign key naming in configuration * chore: add migrations for reset password token table * chore: modified service a little bit - Changing the way service registrations work so that 2 interfaces share the same instance - Pass the whole entity to event so that auth db context can get its id - I forgot the db context model snapshot * chore: tweaks how tokens are sent - send the token instead of the hashed version of it * add: endpoint and service to reset password
* test: add unit test for password hashing * feat(SecurityUtil.cs): add password hashing implementation * refactor: add salt and pepper for password hashing * fix(IdentityService.cs): remove unused dependencies
* update: add audit and private status to document when create new import request * add: get all issued documents * add: checkin endpoint * add: approve and reject endpoint for document * finish: almost everything * reason * rename endpoint
* feat: get self documents * fix: minor stuff --------- Co-authored-by: Nguyen Quang Chien <87883163+ChienNQuang@users.noreply.github.com>
kaitoz11
commented
Jun 21, 2023
Member
Tests are dead! But we don't give a shit! Let's pray for our life |
…e-BE into refactor/quadruple-programming
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.