Fix generated OpenAPI contracts

[byronellis]

Summary

Fixes an issue with openapi generation introduced in PR 737 I believe. It was preventing me from doing a full test cycle, but I believe this PR should fix it. Putting it at the top of my stack for a quick merge.

Changes

• Client-facing API Changes
• Internal API Changes
• Bug Fixes
• New Features
• Performance Improvements
• Code Style
• Refactoring
• Documentation
• Tests

For all the boxes checked, please include additional details of the changes made in this pull request.

Testing Done

• Manually Tested on local docker setup. Please include commands ran, and their output.
• Added new tests for the changes made.
• Updated existing tests to reflect the changes made.
• No tests added or updated. Please explain why. If unsure, please feel free to ask for help.
• Some other form of testing like staging or soak time in production. Please explain.

For all the boxes checked, include a detailed description of the testing done for the changes made in this pull request.

The thing that broke was the testing itself so the fact that they now pass is was acceptance criteria. In future people should probably run tests before merging to avoid breaking other people.

Additional Information

• Breaking Changes
• Deprecations
• Large PR broken into smaller PRs, and PR plan linked in the description.

For all the boxes checked, include additional details of the changes made in this pull request.

[mkuchenbecker]

LGTM, other than making sure we don't document that null=table.

These two tasks should fall on the view and lock projects that are in-flight. I think those projects should ultimately be responsible going forward. If we are seeing more of these issues that block I want to make sure these issues are caught by the build.