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
Discovery indexing user verification #4035
Conversation
@@ -133,3 +149,31 @@ def update_user(params: ManageEntityParameters): | |||
params.add_user_record(user_id, user_record) | |||
|
|||
return user_record | |||
|
|||
|
|||
def verify_user(params: ManageEntityParameters): |
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.
does the user signup flow cover send a verify user tx? i guess all create_user txs have is_verified false to start
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.
Well the user signup flow covers associating instagram or twitter - which is done with identity service.
When that happens, we check in identity if the user is verified and identity sends the request
* Add veirfy user action * lint * Address comments
Description
Add User verified action via entity manager
Tests
Wrote a test to validate that a valid tx updates the user verified and an invalid tx does not update the user model
Monitoring - How will this change be monitored? Are there sufficient logs / alerts?