Skip to content

Resolves #123: Generalize display name to include wards; dedupe juris… - #126

Open
shelltr wants to merge 1 commit into
openstates:mainfrom
shelltr:add-ward-display-name
Open

Resolves #123: Generalize display name to include wards; dedupe juris…#126
shelltr wants to merge 1 commit into
openstates:mainfrom
shelltr:add-ward-display-name

Conversation

@shelltr

@shelltr shelltr commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Change Type

  • YAML data change (divisions / jurisdictions)
  • Code change

Summary

Division YAMLs for ward: OCDids are named without a ward designation or number — the ward number never enters display_name, so the filename collapses to just the place name and a UUID.

This change resolves that issue by handling the ward segment.

As well, moves derive_jurisdiction_ocdid functionality to jurisdiction_seed.py file -- we are duplicating some code there.


Code Change

Linked Issue: Closes #123

Validation:

  • .venv/bin/python -m pytest tests/src/init_migration/test_generate_division.py
  • .venv/bin/python -m src.init_migration.main --state oh --force
  • .venv/bin/python -m pytest tests/src/init_migration/test_jurisdiction_seed.py

@shelltr
shelltr force-pushed the add-ward-display-name branch from 6d64f93 to 6b792a5 Compare July 28, 2026 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ward divisions' YAML filenames omit the ward designation and number

1 participant