I don’t think this change should really cause any issues since the field is already marked as optional. Unfortunately, I don’t have a fully working setup locally to run the tests, but this is pretty narrow.
Before Opening a PR
[ ] I tested this using the CLI (e.g., poetry run vaccine-feed-ingest <state>/<site>)
[ ] I ran auto-formatting: poetry run tox -e lint-fix
getmyvax.org is removing the
eligibilityfield from our location model as part of a move solidify a v1 API: https://github.com/usdigitalresponse/univaf/issues/154. That change should land later today.I don’t think this change should really cause any issues since the field is already marked as optional. Unfortunately, I don’t have a fully working setup locally to run the tests, but this is pretty narrow.
Before Opening a PR
poetry run vaccine-feed-ingest <state>/<site>)poetry run tox -e lint-fix