Closed abitdodgy closed 10 years ago
I think it's a good idea, I faced the same issue.
Mergeable?
I think this was already integrated from the looks of it. I can see notification.approved?
and .cancelled?
etc... although not all states are documented.
Maybe not necessary, but perhaps it's useful to have a method that fetches the notification status directly? This is useful if you want to add a text representation in the database, as opposed to the status code.
We can call things like
approved?
,cancelled?
, and so on... but thenotification.status
method returns an int.This is what I do now: