Notification improvements

[andreascreten]

Added

• Support for using a reference on notifications so you track down which notification an event happened on.
• Support for actions and reply on notifications
• Implemented more events handlers for notification events: reply, action, and closed.

---

Goes hand in hand with NativePHP/electron#168

[SRWieZ]

Hi and thanks for the PR.

Did you managed to test this on your application?

I ask because I don't think the current facade permits the creation of buttons on Notifications.

[andreascreten]

Hi and thanks for the PR.

Did you managed to test this on your application?

I ask because I don't think the current facade permits the creation of buttons on Notifications.

Added the missing methods to both the Notification class and the Facade.

[SRWieZ]

Do you have a screenshot of it working ?

Because from the documentation, actions should be converted to NotificationAction object with a type.

[andreascreten]

You are right! I was using the client directly to test locally, and while moving the code to the Notification class I forgot about the format that is expected. Pushed a fix, and here is a screenshot:

For completeness, an example of a reply:

[SRWieZ]

Great!

I will take some time tomorrow to test your PR on my computer.

Can you PR the docs in the meantime?
https://github.com/NativePHP/nativephp.com/blob/main/resources/views/docs/desktop/1/the-basics/notifications.md

[andreascreten]

Done deal!

NativePHP/nativephp.com#83

[PeteBishwhip]

This is some amazing work. I'll also give this a test tomorrow and report back.

Thank you for your contribution!