Code review comment for lp:~linaro-infrastructure/launchpad/notify-workitems-changes

Revision history for this message
Mattias Backman (mabac) wrote :

Removing @block_implicit_flushes worked just fine, all tests pass and nothing seems broken when running locally. Now the diffs look good and we get notifications for all changes without extra commits.

I added code to send notifications from the +workitems page and in the process fixed it so that is also saves the changes.

« Back to merge proposal