Code review comment for lp:~timchen119/notify-osd/notify-osd.lp404658

Revision history for this message
Lars Karlitski (larsu) wrote :

Thanks for this patch, but it doesn't fully fix bug #404658. The summary and body of a synchronous notification are not updated when a new synchronous notification is sent.

This patch also changes whitespace around the changed lines.

I've incorporated your patch in lp:~~larsu/notify-osd/update-sync.

« Back to merge proposal