search

UiL-OTS-labs / ppn-backend

Backend application for the adult participant administration system
https://experimenten-admin.lab.hum.uu.nl
Other
0 stars 0 forks source link

Alle exp zijn voor eentaligen geworden? #90

Closed irisopgithub closed 1 month ago

irisopgithub commented 2 years ago

zowel het exp van tijn als van marijke hebben in de nieuwe database eentalig als criterium, terwijl dat 'maakt niet uit' was in het oude systeem. het viel me op omdat ik opeens veel minder ppn kan uitnodigen voor marijkes exp.

ik zet het nu gewoon met de hand op 'maakt niet uit' (er vinden al mutaties plaats nu in de database, mensen schrijven zich uit voor de mailinglist nav de 'hoera we hebben een nieuwe website' mail). maar dit is denk ik een bugje in de data-overzet-procedure waar je misschien wel even naar wilt kijken.

irisopgithub commented 2 years ago

oh, en de minimum leeftijd is ook niet overgekomen, die staat nu op 'maakt niet uit' (was 18)

irisopgithub commented 2 years ago

max leeftijd ook niet (die was bij tijn 35)

tymees commented 2 years ago

Ik heb het probleem gevonden, en zo te zien zijn alle standaard criteria niet goed overgekomen....

Nou heb ik dat probleem gefixt, maar nu loop ik tegen het probleem aan dat mijn migratie code niet geschreven is om op een volle database te gaan schrijven. (Of uberhaupt 1 onderdeel ervan draaien). Het grootste probleem is dat ik een relatie moet leggen tussen data uit de oude en de nieuwe, wat nu gebeurd door gewoon welk nieuw object bij de oude hoort bij het aanmaken ervan...

Die luxe heb ik nu niet, dus ik moet een andere manier vinden om die relatie te leggen... Een snelle blik lijkt uit te wijzen dat je altijd erg netjes experimenten een andere naam hebt gegeven, dus mogelijk kan ik op naam selecteren

irisopgithub commented 2 years ago

ik stel voor om het zo te laten. de kans dat we nog voor andere bestaande exp gaan uitnodigen is heel erg klein, en als dat wel moet, kan ik altijd uit de beschrijving nog wel halen wat de criteria moeten zijn.

tymees commented 2 years ago

Zeker weten?

Ik heb al een proof-of-concept werken op de test database

irisopgithub commented 2 years ago

zeker weten. ik word extreem zenuwachtig van het idee om in de huidige database dingen te gaan veranderen, en het is echt niet nodig.

irisopgithub commented 1 month ago

ik weet niet of je hier nog iets mee wilt voor een toekomstige migratie?

tymees commented 1 month ago

Nee, dit is alleen een probleem voor de migratie van de oude naar de nieuwe; Verdere migraties gebruiken een ander (minder foutgevoelig) systeem van Django zelf.

In theorie kan dit voor babex een ding zijn, gezien die gebasseerd is op PPN, maar gezien die oude DB totaal anders is moet daar nieuwe migratie code voor geschreven worden.

Ik sluit deze issue dan maar ook, het is niet relevant meer