{"action":"created","issue":{"url":"https://api.github.com/repos/karpenoktem/kninfra/issues/359","labels_url":"https://api.github.com/repos/karpenoktem/kninfra/issues/359/labels{/name}","comments_url":"https://api.github.com/repos/karpenoktem/kninfra/issues/359/comments","events_url":"https://api.github.com/repos/karpenoktem/kninfra/issues/359/events","html_url":"https://github.com/karpenoktem/kninfra/pull/359","id":128350158,"number":359,"title":"[WIP] Afmelden","user":{"login":"aykevl","id":729697,"avatar_url":"https://avatars.githubusercontent.com/u/729697?v=3","gravatar_id":"","url":"https://api.github.com/users/aykevl","html_url":"https://github.com/aykevl","followers_url":"https://api.github.com/users/aykevl/followers","following_url":"https://api.github.com/users/aykevl/following{/other_user}","gists_url":"https://api.github.com/users/aykevl/gists{/gist_id}","starred_url":"https://api.github.com/users/aykevl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aykevl/subscriptions","organizations_url":"https://api.github.com/users/aykevl/orgs","repos_url":"https://api.github.com/users/aykevl/repos","events_url":"https://api.github.com/users/aykevl/events{/privacy}","received_events_url":"https://api.github.com/users/aykevl/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2016-01-23T20:20:53Z","updated_at":"2016-01-31T19:30:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/karpenoktem/kninfra/pulls/359","html_url":"https://github.com/karpenoktem/kninfra/pull/359","diff_url":"https://github.com/karpenoktem/kninfra/pull/359.diff","patch_url":"https://github.com/karpenoktem/kninfra/pull/359.patch"},"body":"**Nog niet mergen!**\r\n\r\nFunctie zodat je je ook kunt afmelden zolang de activiteit nog open is.\r\nWat nog gedaan moet worden:\r\n * zorgen dat oude events ook de juiste mail templates hebben\r\n * een volledige geschiedenis van subscribe/unsubscribe acties weergeven\r\n * meer testen\r\n\r\nDit is vooral een eerste opzet voor review.\r\n![screenshot-3](https://cloud.githubusercontent.com/assets/729697/12532447/d3e2d006-c214-11e5-84c7-af245ca55af4.png)\r\n\r\nEen mogelijke extra feature zou een minimum aantal deelnemers kunnen zijn: als het aantal daaronder zit, kun je je niet (meer) afmelden. Voor activiteiten die dat nodig hebben.\r\n\r\nVooral bij die e-mail templates denk ik dat het anders moet.\r\nDe e-mail templates worden nauwelijks aangepast. Hoogstens een keer een kleine grappige aanpassing. Dus zo belangrijk is die mogelijkheid tot aanpassen niet.\r\nVerder vind ik ze nogal beperkt. Het zijn tekstmails (geen HTML), het zijn geen Django templates waardoor ze minder flexibel zijn, en de 'templates' staan in Python code wat minder mooi is.\r\n\r\nMijn voorstel:\r\nHerschrijf het huidige systeem volledig. Gebruik niet meer de oude Python string templates, maar maak mooie HTML templates. E-mails kunnen dan zowel in HTML als plain text verzonden worden, via eenzelfde systeem als voor de informacie gebruikt wordt (HTML naar Markdown converteren).\r\nDe voordelen van Django templates en HTML (die ik zo snel zie) zijn dat we bijvoorbeeld echte links kunnen gebruiken en wat logica, waarmee we dus niet per sé een leeg \"opmerkingen:\" stuk hebben. Verder zijn ze in elk geval een stuk makkelijker aan te passen (in code) als we in de toekomst dit soort updates hebben.\r\n\r\ncloses #151 "},"comment":{"url":"https://api.github.com/repos/karpenoktem/kninfra/issues/comments/177594162","html_url":"https://github.com/karpenoktem/kninfra/pull/359#issuecomment-177594162","issue_url":"https://api.github.com/repos/karpenoktem/kninfra/issues/359","id":177594162,"user":{"login":"aykevl","id":729697,"avatar_url":"https://avatars.githubusercontent.com/u/729697?v=3","gravatar_id":"","url":"https://api.github.com/users/aykevl","html_url":"https://github.com/aykevl","followers_url":"https://api.github.com/users/aykevl/followers","following_url":"https://api.github.com/users/aykevl/following{/other_user}","gists_url":"https://api.github.com/users/aykevl/gists{/gist_id}","starred_url":"https://api.github.com/users/aykevl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aykevl/subscriptions","organizations_url":"https://api.github.com/users/aykevl/orgs","repos_url":"https://api.github.com/users/aykevl/repos","events_url":"https://api.github.com/users/aykevl/events{/privacy}","received_events_url":"https://api.github.com/users/aykevl/received_events","type":"User","site_admin":false},"created_at":"2016-01-31T19:30:42Z","updated_at":"2016-01-31T19:30:42Z","body":"E-mails zijn nu gedaan. Een kleine aanpassing: de extra lege regel na \"Met een vriendelijke groet,\" heb ik verwijderd. En ik heb alles omgezet in HTML (met tekst-fallback).\r\nEen issue in HTML2Text: https://github.com/Alir3z4/html2text/issues/114 (alleen relevant voor text/plain en opmerkingen over meer dan 1 regel).\r\n\r\nIk heb er wat mee rond gespeeld en het lijkt allemaal goed te werken. E-mail kan ik niet direct testen: ik heb daarvoor de code wat aangepast om het bericht op stdout te printen in plaats van te mailen, en die tekst ziet er goed uit. Dit is een vrij grote verandering (groter dan ik had verwacht) en dus lastig goed te testen.\r\nIs dat genoeg getest?"}}