Re: [PATCH 0 of 9] remove UI notification feature

2018-12-06 Thread Mads Kiilerich
On 12/6/18 1:02 PM, Thomas De Schampheleire wrote: And we have so much cruft in upgraded databases anyway. I think we either should accept a bit more tech debt there and rely on a future "only preserve schema things we use now" database migration option, or just add a migration step now to drop

Re: [PATCH 0 of 9] remove UI notification feature

2018-12-06 Thread Thomas De Schampheleire
El jue., 6 dic. 2018 a las 12:26, Mads Kiilerich () escribió: > > Thanks. Thanks for pushing so quickly. > > I wonder if some css also could be cleaned up? Do you have specific examples or details? I searched for 'notif' but found nothing. > > And we have so much cruft in upgraded databases

[PATCH 0 of 9] remove UI notification feature

2018-12-06 Thread Thomas De Schampheleire
Hello, As discussed, this patch series removes the UI notification feature from Kallithea. Notifications are now email-only. This change simplifies the code base and will make later changes in the email notifications easier. The first three commits are not directly related to the removal, but