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 geimporteerde ppn hebben een secondary emailadres (dat leeg is) #110

Closed irisopgithub closed 2 years ago

irisopgithub commented 2 years ago

Dit is niet erg, maar als je je importscript nog eens wilt gebruiken wel handig om te weten: alle geimporteerde ppn hebben nu een (vaak leeg) secondary emailadress. Ik ontdekte het doordat dat een verplicht veld is als je een pp edit (ben wat bouncers weg aan het werken). Je kunt het secondary emailadres dan ook deleten dus het is in de praktijk geen probleem.

tymees commented 2 years ago

Het import script is compleet op maat gemaakt, dus die gaat niet hergebruikt worden.

Ik kan relatief makkelijk alle lege secundaire email adresssen weghalen (en ik heb ze allemaal gecheckt of ze echt leeg zijn), als je dat wilt?

irisopgithub commented 2 years ago

ja graag, misschien laadt de pp-pagina dan ook wat sneller, hij is vrij traag nu

tymees commented 2 years ago

Ze zijn weg, maar ik denk niet dat dat heel erg veel scheelt. De traagheid komt van 3000+ rijen moeten tonen en sorteren.

Dat kan wel slimmer gedaan worden, maar dan moet sortering e.d. allemaal op de server gebeuren. We waren al met zoiets bezig voor de FETC, maarja, tijdgebrek en urgente problemen....

irisopgithub commented 2 years ago

het scheelt idd niks qua laadtijd, maar het ziet er wel netter uit, en het scheelt me ook werk bij het mergen van ppn. dus toch fijn.