Implement `did last [monday..sunday]` #342

xsuchy · 2023-11-20T11:40:57Z

Fixes: #305

psss

Thanks for extending the day selection. Looks good, only when selecting the last day which is the same as today results in today's report:

> date
Tue Dec  5 00:02:37 CET 2023

> did last tuesday
Status report for the last tuesday (2023-12-05 to 2023-12-05).
...

Could you please cover that use case as well? Thanks.

psss · 2023-12-13T22:04:11Z

Seems the last adjustment broke some intervals: After the change we get one week before the expected interval:

> date
Wed Dec 13 23:01:05 CET 2023

> did last friday
Status report for the last friday (2023-12-01 to 2023-12-01).

Failing tests confirm the problem.

xsuchy · 2024-03-25T10:23:00Z

Ping. Anything blocking this PR?

psss

Now looks good. Thanks!

psss changed the title ~~implement last [monday..sunday]~~ Implement the last [monday..sunday] Dec 4, 2023

psss changed the title ~~Implement the last [monday..sunday]~~ Implement did last [monday..sunday] Dec 4, 2023

psss added the base label Dec 4, 2023

psss added this to the 0.22 milestone Dec 4, 2023

psss self-assigned this Dec 4, 2023

psss requested changes Dec 4, 2023

View reviewed changes

xsuchy force-pushed the monday branch from 4ebefa7 to b2c2ff6 Compare December 5, 2023 08:12

xsuchy force-pushed the monday branch from b2c2ff6 to 989311b Compare February 10, 2024 16:58

Implement did last [monday..sunday]

cf62044

Fixes: psss#305

psss force-pushed the monday branch from 989311b to cf62044 Compare March 27, 2024 08:02

psss approved these changes Mar 27, 2024

View reviewed changes

psss merged commit cf62044 into psss:main Mar 27, 2024
16 checks passed

Provide feedback