-
Notifications
You must be signed in to change notification settings - Fork 3
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
Persists user information during role entry and publication requests #79
Conversation
This should cover cases with missing roles as well as third-party.
302763a
to
964fb66
Compare
Looks like a release of openstax-accounts is going to be required. |
The coverage is not going to pass anytime soon. I'm managed to scope the coverage report, but that has brought down the coverage by nearly 2%. The good thing is that it is now more accurate. |
|
||
# Update existing records. | ||
for user_id in existing_user_ids: | ||
# TODO only update based on a delta against the 'updated' column. |
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.
doesn't really save anything: postgresql will need to form an entire new tuple anyway, and the data here is so small not sending the unchanged values isn't worth it. Simple is better. So, no TODO
Persists user information during role entry and publication requests
Depends on openstax/openstax-accounts#16 & openstax/cnx-archive#290
See also trello card