finalize user admission priorities for backend #987
Conversation
magsyg
changed the title
magsyg
force-pushed
the
feature/backendpriority
branch
from
947dda8
to
7c90f0b
Compare
|
|
||
| """ | ||
| admissions_for_user = RecruitmentAdmission.objects.filter(recruitment=self.recruitment, user=self.user) | ||
| # Use order for more simple an unified for direction |
There was a problem hiding this comment.
Hva er det du ikke skjønner?
Får tak i alle admissions for bruker og recruitment
There was a problem hiding this comment.
Mistenker at man kan gjøre dette enklere, men det avhenger av noen ting sikkert.
Er det validering på at man ikke kan ha samme priority, er vi sikret mot feil tilstand?
There was a problem hiding this comment.
Er høy eller lav prio best? Er det som rekkefølge slik at 1 er øverst?
There was a problem hiding this comment.
Lav, er mer rangering
There was a problem hiding this comment.
Gjerne svar på alle spørsmålene. Skal komme opp med et forslag, tror jeg har en god ide.
There was a problem hiding this comment.
Den er sikret mot at gjennom bruk av denne, så kan ikke flere ha samme rangering, siden den vill bytte opp plassering og sortere.
Den PR er lagd for å organisere og omgjøre prioriteringer.
Den kan settes manuelt til samme, men den beskyttelsen vil være i en annen PR.
There was a problem hiding this comment.
Så det er en db constraint på vei inn?
There was a problem hiding this comment.
Det er ikke en DB constraint nei, det kan settes manuellt, og da sier ingenting at det ikke er mulig.
En DB constraint i dette tilfellet kan foresake en form for deadlock, hvor når den bytter om så vil det være et tilfelle 2 har samme verdi, og vil da fucke ting opp.
There was a problem hiding this comment.
(Denne tråden var vi ikke ferdig med.)
Det er litt av poenget med constraint, garantere at ingen av dem har samme prio.
Jeg tror dette kan gjøres mye enklere.
Bare bulk flytt på settet du finner.
qs.filter(...).update(prio=F('prio')+1)
qs.filter(...).update(prio=F('prio')-1)
frontend/src/Pages/ApplicantApplicationOverviewPage/ApplicantApplicationOverviewPage.tsx
Outdated
Show resolved
Hide resolved
frontend/src/Pages/ApplicantApplicationOverviewPage/ApplicantApplicationOverviewPage.tsx
Outdated
Show resolved
Hide resolved
magsyg
force-pushed
the
feature/backendpriority
branch
from
af560da
to
57d477f
Compare
closes #984