Page MenuHomePhabricator

Review and expand code documentation for users
Closed, ResolvedPublic

Description

In the users application, review Python code and templates and add documentation where it is missing or incomplete. This might include in-line comments, docstrings, and/or more verbose code.

Event Timeline

A couple of notes I made while going through this that I forgot to post until now. They can be followup tasks or I could wrap any necessary work into this PR:

  • Do we still need authorization_backfill.py? I think this was a one-time script.
  • There's a TODO in user_check.py - is there a Phab task for this or shall I file one?

Do we still need authorization_backfill.py? I think this was a one-time script

I don't think we do.

There's a TODO in user_check.py - is there a Phab task for this or shall I file one?

I'm not sure we use this management command anymore.

Resolving this because a follow-up ticket has been created. Thanks, @Samwalton9!