Only update noty.sent_at on successful write #55

Open
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants

barce commented Sep 27, 2012

Hi,

Thanks for extending and keeping this gem up-to-date.

Would you please accept this pull request that only updates noty.sent_at on a successful write?

I ran into an issue where notifications were marked as sent when they were not.

Cheers, Barce

+1 for the idea

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment