yournextrepresentative icon indicating copy to clipboard operation
yournextrepresentative copied to clipboard

migrate further fields from ResultEvent to use django-popolo models

Open mhl opened this issue 9 years ago • 1 comments

Of the fields in ResultEvent, we've already migrated winner to be a foreign key to Person, but there are other updates that could be made to improve data integrity:

  • winner_person_name isn't needed any more - winner.name would work
  • post_id and post_name can be replaced by post (a foreign key to Post) and post.label
  • winner_party_id can be replaced by winner_party (a foreign key to Organization)
  • The winner_party_name property method can be replaced by winner_party.name

mhl avatar Dec 16 '15 19:12 mhl

These changes have been made in the DC fork, and could be backported by cherry-picking

mhl avatar Aug 14 '17 14:08 mhl