[O] [PATCH] org-notify: Fix compatibility with latest org-element (uppercase properties)

[Peter Münster]

* contrib/lisp/org-notify.el (org-notify-make-todo): Properties from
  org-element are all uppercase now.
* (org-notify-maybe-too-late): Fix typo in docstring (thanks to davemq,
  https://github.com/p-m/org-notify/issues/3).
---
 contrib/lisp/org-notify.el | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/contrib/lisp/org-notify.el b/contrib/lisp/org-notify.el
index da2c96f..8bcca9f 100644
--- a/contrib/lisp/org-notify.el
+++ b/contrib/lisp/org-notify.el
@@ -120,7 +120,7 @@ simple timestamp string.
   Create one todo item.
   (macrolet ((get (k) `(plist-get list ,k))
  (pr (k v) `(setq result (plist-put result ,k ,v
-(let* ((list (nth 1 heading))  (notify (or (get :notify) default))
+(let* ((list (nth 1 heading))  (notify (or (get :NOTIFY) default))
(deadline (org-notify-convert-deadline (get :deadline)))
   (heading (get :raw-value))
result)
@@ -148,7 +148,7 @@ simple timestamp string.
  'headline 'org-notify-make-todo)
 
 (defun org-notify-maybe-too-late (diff period heading)
-  Print waring message, when notified significantly later than defined by
+  Print warning message, when notified significantly later than defined by
 PERIOD.
   (if ( (/ diff period) 1.5)
   (message Warning: notification for \%s\ behind schedule! heading))
-- 
2.1.4

   Peter

Re: [O] [PATCH] org-notify: Fix compatibility with latest org-element (uppercase properties)

[Nicolas Goaziou]

Hello,

Peter Münster pmli...@free.fr writes:

 * contrib/lisp/org-notify.el (org-notify-make-todo): Properties from
   org-element are all uppercase now.
 * (org-notify-maybe-too-late): Fix typo in docstring (thanks to davemq,
   https://github.com/p-m/org-notify/issues/3).

Applied. Thank you.

Out of curiosity, as org-notify.el's author, don't you have write access
into repository? If that's the case, you don't need to wait for review.


Regards,

-- 
Nicolas Goaziou

Re: [O] [PATCH] org-notify: Fix compatibility with latest org-element (uppercase properties)

[Peter Münster]

On Mon, Aug 24 2015, Nicolas Goaziou wrote:

 Out of curiosity, as org-notify.el's author, don't you have write access
 into repository?

I guess no, but I've never tried...

-- 
   Peter