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

Set sender of emails to new NPM #123

Closed tymees closed 4 months ago

tymees commented 10 months ago

Still needs to be updated on the privacy policy and the subscribe pages; (but keeping the admin name)

In addition, develop changes need to be applied to the DSC 3 version as well...

irisopgithub commented 4 months ago

Ik weet niet wat je precies bedoelde met deze issue, maar op https://experimenten.acc.lab.hum.uu.nl/privacy/ zou ik 'taal experimentenmailadres' graag willen vervangen door mijn naam, of (ook prima als dat makkelijker is), zoals op https://experimenten.acc.lab.hum.uu.nl/participant/sign_up/, gewoon de tekst taalexperimenten@uu.nl. Maar misschien is dat ook wat je bedoelt?

tymees commented 4 months ago

Nee dat is niet precies wat ik bedoelde.

Om eerst even de naam situatie uit te leggen: Op de acc-server had ik, als test, een local-admin aangemaakt met de naam 'taal experimentenmailadres' (en dus de NPM als email). Deze local-admin is daar ook de 'hoofd admin'.

Die twee pagina's gebruiken dan ook de gegevens van de 'hoofd admin', met het verschil dat de privacy pagina de naam als display-waarde gebruikt en de sign-up pagina daar de email als display-waarde gebruikt. Vandaar dit verschil. (Ik kan me ook niet bedenken wat hier de reden voor was).

Dus concreet bedoelde ik dat ik dat die twee pagina's moet veranderen zodat daar het NPM adres in de 'mailto' moet komen, ipv het adres van de admin. (Maar dus wel de 'admin' naam als display-waarde te blijven gebruiken).

Nou brengt dit wel de vraag op wat er moet gebeuren met de sign-up pagina. Moet dat een e-mail adres blijven, of toch de naam van de admin (op productie dus jouw naam)? Beide opties zijn even moeilijk om te doen ;) (Vrij simpel trouwens)

irisopgithub commented 4 months ago

wat mij betreft de naam van de admin, op beide pagina's. er is al zoveel in de wereld aan naamloze mailboxen en formulieren, en daar wordt volgens mij niemand gelukkiger van. dus waar we de suggestie van de menselijke maat kunnen wekken doe ik dat graag.

tymees commented 4 months ago

Okay!

Ik heb die wijziging nu doorgevoerd in de code