[bug]: 'thisweek' doesn't work in the date parser about ultralist HOT 3 OPEN

I am looking into implementing this in my fork, Ultodo, but here is how I worked around it:

I have an alias:
_yesterday='echo Thu | tr '''[:upper:]''' '''[:lower:]''

that gets me the abbreviated day name in lowercase...

Then another alias:

alias week="ultralist l duebefore:$(_yesterday)"

This is a bit convoluted, but I needed yesterday because using today results in showing the 7 days up to today, since it uses closest monday. Using yesterday gets us this week.

I hope to implement thisweek soon so that aliases are not needed.

Thanks

from ultralist.

@sottey will your fix make lastweek cover the previous 7 days? Because I think it'd be kind of odd -- a week is normally thought of as the set of 7 different consecutive days of the week (Monday - Sunday or Sunday - Saturday) depending on locale and convention, not any 7 days.

It makes more sense to have the week start on Monday -- then thisweek is any day between the most recent (including today) Monday and the closest upcoming (including today) Sunday, lastweek is the week starting the Monday before most recent, and nextweek is the week starting the Monday after the closest upcoming Sunday.

Take a look at my fix at #259 ; I've also added the monthly options -- and by comparison, this will make weekly ones standout -- nobody thinks "last month" as the 30 days before today -- it's assumed to be the calendar month, regardless of where in the current month we are.

from ultralist.

Hi @ltrubov ,

Yeah, that was what I did. I feel like "7 days from the most recent past Monday" makes the most sense. Your work in the input_parser was more elegant than mine so I incorporated it, thanks!

LOVE the month stuff you did. I have incorporated that as well. Thanks!

from ultralist.