Skip to content

Protect endpoints that require JSON requests

Allan Shortlidge requested to merge require-json-for-member-endpoints into dev

@DSASanFrancisco/portal-members

Use the requires_json decorator to protect each endpoint in members.py, meetings.py, and elections.py that requires a JSON formatted request. Add missing tests for election candidate endpoints.

SUMMARY

Merge Request Rules

Author

  • If the MR isn't finished, prepend "WIP" to the title.
  • Unless otherwise noted, you should merge after approval.

Reviewer

  • Comments aren't necessarily blockers but can be. Add [blocker], or something similar, to a comment as necessary, otherwise the author may not know what's a blocker and what's not.
  • Bias toward approving the MR and ask for follow-up work as needed. You can make these issues as needed.
  • Be proactive about approving. If you and other reviewers make comments, and yours are resolved by the author, use your judgement to either approve or follow up with the other reviewers. Again, bias toward approving.
Edited by Allan Shortlidge

Merge request reports