fix postitions can have gangs from seperate organizations #1636
Conversation
There was a problem hiding this comment.
Det ligger også gjenger på opptak som ikke er "eid" av organisasjonen som "eier" opptaket.
Det finnes altså f.eks. en side for "Økonomi" i et Samf opptak. Se http://localhost:3000/control-panel/recruitment/1/gang/23
Bare å bytte ut gjeng id (den siste) i URLen. Alle over 17 er ikke Samf gjenger.
Så problemet du løser i denne PR-en burde i tillegg løses på et nivå over hvor du gjør det her.
| if self.gang and self.section: | ||
| errors['gang'].append(self.ONLY_ONE_OWNER_ERROR) | ||
| errors['section'].append(self.ONLY_ONE_OWNER_ERROR) | ||
| elif not (self.gang or self.section): | ||
| errors['gang'].append(self.NO_OWNER_ERROR) | ||
| errors['section'].append(self.NO_OWNER_ERROR) | ||
| raise ValidationError(errors) |
There was a problem hiding this comment.
Noe av dette blir vel gjort i #1632 også
damn nice spotta |
Found a bug where a recruitment can have positions from gangs from other organizations, such as an UKA position in a Samf recruitment
These would not appear in the admin panel