-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feat/verify #275
Feat/verify #275
Conversation
Coverage report
Test suite run success255 tests passing in 22 suites. Report generated by 🧪jest coverage report action from f435cba |
I have deployed this PR to feat-verify-ekorre.review.esek.se 🚀 |
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.
add some tests for your new api functions pls
Made changes after review! |
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.
This has been reviewed before :)
LGTM! Reqest solved :) |
I have removed this deploy now 😇 |
Added verification logic to user using a VerifyInfo object for future expansion.
Added connection to verify microservice.
Added automatic verification if user has a post.
!! - - This update is dependent on the same feature in Efterphest and Auth. - - !!