Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Workers have to be duplicated for each society #4

Closed
thor opened this issue Mar 9, 2017 · 0 comments
Closed

Workers have to be duplicated for each society #4

thor opened this issue Mar 9, 2017 · 0 comments
Assignees
@thor
Labels
enhancement major

Comments

@thor
Copy link
Member

thor commented Mar 9, 2017

Unfortunately our OneToOneField usage limits each worker to one society, which means these workers that are active in more than one society have to be duplicated. This kills the normalisation.

  • Change to ManyToManyField.
  • Create a manual migration to migrate back to them.
@thor thor self-assigned this Aug 17, 2017
@thor thor removed the help wanted label Sep 4, 2017
@thor thor added the major label Oct 26, 2017
thor added a commit that referenced this issue Oct 31, 2017
@thor
MAGICAL ONE-WAY MIGRATION FUNCTION FOR DUP-WORKERS
c9dec43 
This does magic, and can for the most be said that it fixes #4.

There are other issues related to invoicing that popped up after this,
but this did actually work with our current production data, so... yay?

The code needs to be friendlier to read though.
@thor thor closed this as completed in 76bd1fd Nov 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thor thor

Labels
enhancement major
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@thor