akhramov / org-wild-notifier.el

Alert notifications for org-agenda
GNU General Public License v3.0
149 stars 21 forks source link

time not being extracted #30

Open holtzermann17 opened 4 years ago

holtzermann17 commented 4 years ago

Config:

GNU Emacs 26.3 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.22.30) of 2019-12-03 Org mode version 9.1.9 (release_9.1.9-65-g5e4542 @ /snap/emacs/current/usr/share/emacs/26.3/lisp/org/)

Repro:

I used a minimal .emacs file for testing, after installing the package from MELPA.

(when (>= emacs-major-version 24)
  (require 'package)
  (add-to-list 'package-archives '("melpa" . "http://melpa.milkbox.net/packages/") t)
  (package-initialize)
  )

(require 'org-wild-notifier)
(org-wild-notifier-mode t)
(setq org-wild-notifier-alert-time 1)

(setq org-agenda-files '("~/org-files/"))
(setq org-agenda-diary-file "~/org-files/work.org")

I have tried to set up notifications for a repeating event like this (in work.org)

* Repeating
** TODO 16:00 Attend Standup 
   SCHEDULED: <%%(let ((dayname (calendar-day-of-week date))) (memq dayname '(1 2 3 4 5)))> 
   :PROPERTIES:
   :WILD_NOTIFIER_NOTIFY_BEFORE: 5
   :END:

I can't get the package to emit notifications. When I look at a trace, it seems the "time" data is not being extracted properly.

1 -> (org-wild-notifier--check-event ((times nil) (title . "16:00 Attend Standup") (intervals 10 5)))
1 <- org-wild-notifier--check-event: nil
akhramov commented 4 years ago

Yeah, it is not supported yet, unfortunately.

fwiw this line needs to be extended https://github.com/akhramov/org-wild-notifier.el/blob/713c5205869dde4d42127fd9365f5831ec222503/org-wild-notifier.el#L271

xzebra commented 4 years ago

This also happens with repeated tasks format, e.g. <2020-09-17 17:50-20:50 .+1w> is parsed by org-parse-time-string as if it was <2020-09-17 17:50>.

Do you have any idea how could this be done? I think this type of tasks are a very important part of org-agenda.