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

Mergen van ppn en meerdere emailadressen #111

Closed irisopgithub closed 2 years ago

irisopgithub commented 2 years ago

Als je twee ppn merget, gebeurt er dit:

van de nieuwe pp krijgen we: de naam telnr of ze student zijn of niet

(oftwel, de dingen die kunnen veranderen - ik neem aan dat van veranderbare special criteria we ook die krijgen van de nieuwe pp)

van de oude pp komt over: email (primair emailadres) dysl moedertaal geboortedatum links/rechtshandig geslacht of de pp op de mailinglist staat - dat klopt eigenlijk niet, zou ik zeggen, dat is een veld dat kan veranderen, en daarvan zou je de nieuwste waarde willen pakken. maar ik de praktijk maakt het niet uit want ik kijk natuurlijk toch eerst op welke mailadressen ze gemaild willen worden, en als ze ergens mailen op 'nee' hebben staan haal ik die natuurlijk uit de uiteindelijke, gemergede pp.

Waar ik nu mee zit, is dat als ik een uitnodigingsmail stuur aan een pp met meerdere mailadressen, dan gaat die alleen naar het primaire mailadres, niet naar alle mailadressen voor die pp. Dat was in het oude systeem ook zo, maar ik kwam daar pas jaren later achter, toen het nieuwe systeem al gebouwd werd.

Ik heb zelf dus altijd gedacht dat ppn met meerdere emailadressen, op alle adressen uitnodigingen en bevestigingen en reminders kregen. Dat leek mij logisch, ze gebruiken meerdere adressen, ok prima, dan doen wij dat ook. Ik snap op zich de logica wel van alleen het primaire mailadres gebruiken, dat is overzichtelijker. Maar als we dat doen, moeten we wel het meest recente mailadres gebruiken natuurlijk. Niet het oude mailadres. Dus dan moet in de merge het nieuwe adres het primaire emailadres worden, niet het oude.

tymees commented 2 years ago

Vanaf de volgende update wordt de email van 'nieuw' de primaire mail van de bestaande, en wordt de oude bestaande email een secondair adres