jariz / Noti

Receive Android notifications on your mac. (w/PushBullet) ⛺
MIT License
911 stars 63 forks source link

Added basic support for API push #43

Closed ABeltramo closed 7 years ago

ABeltramo commented 7 years ago

It's working for my use case (see issue 42 ) i don't know if it's the solution to also issue 21. For sure it's a starting point to add other useful notification to Noti. The line now -= 5 it's a little trick and I wan't to find a better code when I have the time. Hope to help others! Good work btw..

ABeltramo commented 7 years ago

As I said it's a starting point 😃 What do you propose to do on click? For example I use it to get notification from a server machine not to send link. Autodetect content? For the -= 5 hack I have to take better look at the API. That's the first simple solution to check if my code works.

jariz commented 7 years ago

Pushes with a link have a optional 'url' property (see the docs on pushes), so that would be really easy to use. 😄

ABeltramo commented 7 years ago

I made a few changes on the code.

Todo (need help)

jariz commented 7 years ago

Looks good :)
I'll see if I can help you with those other points this week.

ABeltramo commented 7 years ago

Little update: both message sent from app and from website with encryption activated it's plain text. Seems we don't need to decrypt for this kind of message. Can you confirm this?

ABeltramo commented 7 years ago

Everything it's completed, I think it's ready to be merged! 🏁

cbarratt commented 7 years ago

Was looking to see whether Noti had this functionality! Looks good, waiting for a release on this :)

ABeltramo commented 7 years ago

Closing this. We are refactoring in Swift 4