- Issue created by @lucas.vidaguren
- Merge request !30Set the next and previous date starting on monday instead of +/-7 days from now → (Closed) created by lucas.vidaguren
- First commit to issue fork.
- Status changed to Needs review
10 months ago 7:32pm 7 April 2024 Thank you for reporting this issue.
@lucas.vidaguren Could you please test if the situation is fixed by the changes I've added in this MR ?
Looking forward to hearing from you
Hi @matthieuscarset I just saw your message. Later today we will test it, by seeing the changes I'm pretty sure MR !31 will fix the changes. I'd proposed a very similar thing on MR !30, but using Carbon instead of DateTime. Your fix seems cleaner.
As soon as we test it I'll give you feedbackIt almost works, the prev:
- Is going 2 weeks before (only in configuration using UTC-3, I think "last day last week" is being iterpretated somehow the wrong way)
- Does't bring the last week events with the view filtering from the $now day (as it shows the week but loads the events from last day to last day)
I think it would work better with "fist day last week" on prev modify
- Status changed to Fixed
6 months ago 4:34pm 17 August 2024 Thank you for your help testing this issue and please forgive the delay in answering.
I've tested again and there was indeed an issue with the previous date when using
last day of last week
.I've merged with the change you suggested:
first day of last week
.It's on 2.1.x and will be release soon.
Automatically closed - issue fixed for 2 weeks with no activity.