BDMS 389: add notes to contact

[jacob-a-brown]

Why

This PR addresses the following problem / context:

• well-inventory-csv.feature requires notes to be added to the Contact model

How

Implementation summary - the following was changed / added / removed:

• Added the NotesMixin to the Contact model
• Updated add_contact to includes notes.
  • The notes need to be refreshed after they have been added so that the notes associated with the returned contact object have the correct note_type. If not refreshed their note_types will be members of the enum, not strings.
• Updated tests
• Added Communication as a note_type since that is specified in the feature file

Notes

Any special considerations, workarounds, or follow-up work to note?

• Updated note_type in CreateNote and NoteRersponse schemas to be typed to the NoteType enum, which is made from the lexicon
• This PR is being opened into staging instead of well-inventory-csv to keep this refactor separate from its implementation. If/once it is approved I'll then merge that into well-inventory-csv and make the requisite updates

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Files with missing lines	Coverage Δ
core/enums.py	100.00% <100.00%> (ø)
db/contact.py	100.00% <100.00%> (ø)
schemas/contact.py	100.00% <100.00%> (ø)
schemas/notes.py	100.00% <100.00%> (ø)
services/contact_helper.py	100.00% <100.00%> (ø)
tests/conftest.py	86.39% <100.00%> (+0.09%)	⬆️
tests/test_contact.py	100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

[jacob-a-brown]

I realized that this PR should go into well-inventory-csv with the implementation of the features. I'm going to close it, update the /well-inventory-csv endpoint to correspond with this note implementation, and then open up a PR to the well-inventory-csv branch.

[jirhiker]

@jacob-a-brown Fyi you can edit the base branch of a pr instead of closing and reopening

[jacob-a-brown]

I'll do that next time instead of closing and opening