Org-alert for windows 10 style notifications

[Horrih]

Hello,
Thank you for your amazing package !
At work I have to use windows 10, and I was sorely missing desktop notifications for my events/tasks.
alert, which org-alert uses under the hood, has no such default style available to my knowledge.

I'm probably not representative of your community of users, but would you consider adding a windows example in your README ?
To keep it concise, it could be a simple link to this issue. It could have saved me quite a bit of time figuring it out !

Here is what I ended up doing in my init.el

(use-package alert
  :config
  ;; Add the windows desktop notifications if on windows
  (when (eq system-type 'windows-nt)
    (alert-define-style
     'windows-desktop-notification-style
     :title "Windows Desktop Notification style"
     :notifier
     (lambda (info)
       (let ((notif-id (w32-notification-notify :title (plist-get info :title) :body (plist-get info :message))))
         ;; Close it after 3 seconds (no new notification can be sent if left unclosed)
         (run-with-timer 3 nil `(lambda() (w32-notification-close ,notif-id))))))
    (setq alert-default-style 'windows-desktop-notification-style)))

(use-package org-alert
  :demand  ; Load it on startup, not lazily
  :config
  (org-alert-enable))

[ntBre]

Thanks for figuring this out! My only hesitation is that this feels like it might belong in the alert repo itself. If they don't already have a Windows style defined, do you think they might accept your code snippet as a PR?

Actually, I see that you opened an issue (jwiegley/alert#106) over there too. Let me know how that goes. If I were them, I'd be glad to accept your code here as a new built-in style. If they don't, I can add something to the README, maybe under a new Troubleshooting heading.

[ntBre]

Hey! I was looking at open issues again and was in the middle of adding your example to the README, but I noticed that alert actually has support for the built-in notifications package that should work on Windows. Could you try setting your alert-default-style to 'notifications and see if that works? I've confirmed that 'notifications works on Linux, but I can't check that it solves your original problem on Windows. Either way I will add something to the README to point people in the right direction.