Skip to content

feat(lexicon): add Anthony Water & Sanitation District and Bramasole Farm organizations - #1057

Open
ksmuczynski wants to merge 1 commit into
stagingfrom
kas-new-organizations-lexicon
Open

ksmuczynski wants to merge 1 commit into
stagingfrom
kas-new-organizations-lexicon

Conversation

@ksmuczynski

Copy link
Copy Markdown
Contributor

Summary

Adds Anthony Water & Sanitation District and Bramasole Farm to the lexicon as organization terms.

Why

This PR addresses the following problem / context:

  • No contact can be saved with either organization until both are lexicon terms.

How

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

  • Added both names as organization terms in core/lexicon.json.

Notes

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

  • This PR has no Jira ticket. The jira-key check fails until someone adds the skip-bdms-check label.
  • Verified. tests/test_lexicon.py and tests/test_seed_prereqs.py ran against a local test database. Running init_lexicon on that database added both names under organization.

Contact.organization is a foreign key to lexicon_term.term, so a
contact's organization must already be a lexicon term. Anthony Water &
Sanitation District and Bramasole Farm are not terms yet. No contact
can be saved with either one as its organization.

This commit inserts both terms in alphabetical order among the other
organization terms.

In this category a definition spells out an abbreviated term, for
example Brazos MDWCA. Neither new name is abbreviated, so each
definition is the same as its term.

init_lexicon inserts only the terms a database is missing. Re-seeding
an existing database adds these two and changes no other term.
Deployed environments get them when someone runs
`oco initialize-lexicon`.
@github-actions

Copy link
Copy Markdown
Contributor

Coverage

✅ 82.02% total — gate is 75%.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant