florisz / BerghAdmin

0 stars 0 forks source link

69 more mail updates #91

Closed reinaldbaart closed 2 years ago

reinaldbaart commented 2 years ago

Voor kennisdeling toch nog een pull request. Mailen werkt nu, de geselecteerde personen worden ingevuld in To, en elke ontvanger krijgt een eigen mail zonder de adressen van andere ontvangers te zien. Ik heb ook alvast een begin van een response handler toegevoegd, om straks de verzendstatus in de database te kunnen zetten,

We kunnen eventueel ook nog validatie in de frontend toevoegen, wat vind jij? En de interface om To adressen toe te voegen is zeer simplistisch. We hebben ook de naam van ontvangers, maar die verliezen we omdat we die omzetten naar een puntkomma gescheiden lijst van alleen mailadressen. Misschien kunnen we daar in een toekomstige versie een betere interface voor bouwen, zodat je makkelijker emailadressen kunt toevoegen/wijzigen/verwijderen, bijvoorbeeld via een list componentje.

florisz commented 2 years ago

ziet er weer mooier uit en consistenter qua functionaliteit. De voorgesteld verbeteringen zijn prima, gewoon een issue aanmaken op het bord en doen! Wordt het weer beter van.