Content managers should see the option 'send to list' in their Newsletter overview

jonge-democraten / website

JD website

https://jongedemocraten.nl

MIT License

6 stars 2 forks source link

Content managers should see the option 'send to list' in their Newsletter overview #144

Closed Pi2048 closed 8 years ago

Pi2048 commented 8 years ago

They currently only see 'View in browser' and 'Send test mail'

trolando commented 8 years ago

I found the error. In hemres/admin.py, line 55 checks for the wrong permission. The permission is tested at two locations, at line 55 to check if someone should see the link, and in views.py to see if the are allowed to execute the function. I modified the latter, but did not think about the check at line 55. Will fix.

trolando commented 8 years ago

Fixed in https://github.com/jonge-democraten/hemres/commit/ec06b59c485b6bd62e69285a0e54bfafacbf44de, moved tag 0.9 and updated on the live demo.