Grouping items using :deadline (before ..) is not working

[cmower]

This issue may be due to my lack of knowledge but any help would be greatly appreciated.

My config

    (let ((one-week-from-today (format-time-string "%Y-%m-%d" (org-read-date nil t "+1w"))))
      (setq org-super-agenda-groups
            '(
              (:name "Overdue"
                     :deadline past
                     :order 0)
              (:name "Due today"
                     :deadline today
                     :order 1)
              (:name "Start Today"
                     :scheduled today
                     :order 2)
              (:name "Within week"
                     :deadline (before one-week-from-today)
                     :order 3))))

This is the output when my agenda is built

Week-agenda (W53):
Monday     28 December 2020 W53
Tuesday    29 December 2020

And in the *Messages* buffer, the following error is thrown

Wrong type argument: stringp, one-week-from-today

I have also tried specifying ((one-week-from-today (org-read-date nil nil "+1w"))) as the varlist in let, but same error is thrown.

I'm at a bit of a loss since, as I understand, one-week-from-today should be a string in the format your documentation recommends.

[pkazmier]

You need to use the BACKQUOTE and COMMA to substitute the value of that variable into your org-super-agenda-groups. Try something like this:

(let ((one-week-from-today (format-time-string "%Y-%m-%d" (org-read-date nil t "+1w"))))
      (setq org-super-agenda-groups
            `(
              (:name "Overdue"
                     :deadline past
                     :order 0)
              (:name "Due today"
                     :deadline today
                     :order 1)
              (:name "Start Today"
                     :scheduled today
                     :order 2)
              (:name "Within week"
                     :deadline (before ,one-week-from-today)
                     :order 3))))

Also, if you have a long-running emacs session, then this won't do what you want either as the one-week-from-today will be static at the value when emacs started. PR #149 solves this for us once the maintainer incorporates.

[cmower]

Thanks @pkazmier, this worked great!

Also, if you have a long-running emacs session, then this won't do what you want either as the one-week-from-today will be static at the value when emacs started. PR #149 solves this for us once the maintainer incorporates.

Not actually a problem for me right now, but thanks for letting me know!