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

Overige proefleiders in mailsjabloon #92

Closed irisopgithub closed 2 years ago

irisopgithub commented 2 years ago

De overige proefleiders staan niet in het mailsjabloon bij de uitnodigingsmail. (De proefleider is Tijn Schmitz. ipv De proefleiders zijn Tijn Schmitz, Panagiota Rassia en Dasja de Leeuw.)

Ik zou zweren dat ik eerder extra leaders er wel bij had zien staan, maar bij nader inzien was ik misschien alleen maar onder de indruk dat je het voor elkaar gekregen had om enkelvoud 'is' te krijgen als er maar 1 proefleider is (want ik heb 11 jaar lang met de hand 'zijn' zitten aanpassen naar 'is' als er maar 1 proefleider was).

Het blijkt trouwens echt enorm fijn en handig te zijn om dat mailsjabloon te kunnen editen!

tymees commented 2 years ago

Ik heb de variable {all_leaders_name_list} toegevoegd aan zowel de uitnodiging als bevestiging sjablonen. Deze zet een mooi geformatte lijst van alle proefleiders, zoals:

  1. Jan Jansen
  2. Jan Jansen en Henk Hendriksen
  3. Jan Jansen, Henk Hendriksen en Ingrid Ingridsen
  4. etc...

Ik zou zweren dat ik eerder extra leaders er wel bij had zien staan, maar bij nader inzien was ik misschien alleen maar onder de indruk dat je het voor elkaar gekregen had om enkelvoud 'is' te krijgen als er maar 1 proefleider is (want ik heb 11 jaar lang met de hand 'zijn' zitten aanpassen naar 'is' als er maar 1 proefleider was).

Klopt! Dat was ook zo, toen het nog geen per-experiment sjabloon was. Toen kon ik dat verhaal vormgeven a.d.h.v. dezelfde code als de webpagina zelf. (Dus was het ook relatief triviaal om 'is/zijn' juist neer te zetten).

Echter, met de sjabloon methode kan ik dat niet meer en doe ik een simpele variable-substitutie. En ik was de overige proefleiders vergeten bij het aanmaken van de variabelen.

Overigens kan ik dat is/zijn ook wel automatiseren weer door dat toe te voegen aan de output van de nieuwe tag. Maar dat is nu dus niet zo ;)