Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LaterJS bug when enum days from Monday #7233

Closed
chuckn0rris opened this issue Jul 28, 2023 · 3 comments
Closed

LaterJS bug when enum days from Monday #7233

chuckn0rris opened this issue Jul 28, 2023 · 3 comments
Assignees
Labels
bug Something isn't working forum Issues from forum OEM OEM customer premium resolved Fixed but not yet released (available in the nightly builds)
Milestone

Comments

@chuckn0rris
Copy link

It works when starts enum from Sun, but doesn't if starts from Mon.

Should work from any weekday

Forum post

we have a calendar configuration like this:

{
  "intervals": [
    {
      "recurrentStartDate": "on Mon, Tue, Wed, Thu, Fri, Sat, Sun at 16:00",
      "recurrentEndDate": "on Mon, Tue, Wed, Thu, Fri, Sat, Sun at 06:00",
      "isWorking": false
    }
  ]
}

from our understanding it should lead to 10 hours per each day (Mon ... Sun) but in reality it shows 10 hours on Monday

@chuckn0rris chuckn0rris added bug Something isn't working premium forum Issues from forum OEM OEM customer labels Jul 28, 2023
@canonic-epicure canonic-epicure self-assigned this Jul 28, 2023
@hgz
Copy link

hgz commented Jul 30, 2023

please also have a look especially at using isWorking = true, these configurations in general do not work correctly, i.e. this does not not work, too, it results in 7 days 24 hours

{ "intervals": [ { "recurrentStartDate": "on Mon at 0:00", "recurrentEndDate": "on Fri at 23:59", "isWorking": true } ] }

{ "intervals": [ { "recurrentStartDate": "on Mon at 0:00", "recurrentEndDate": "on Sat at 0:00", "isWorking": true } ] }

@canonic-epicure
Copy link

@hgz Works as intended? Start date Monday 0:00, end date - Friday 23:59 - whole week is marked as a working time.

@hgz
Copy link

hgz commented Aug 2, 2023

not aware of unspecifiedTimeIsWorking, works as intended

@canonic-epicure canonic-epicure added in progress ready for review Issue is fixed, the pull request is being reviewed and removed in progress labels Aug 7, 2023
@isglass isglass added resolved Fixed but not yet released (available in the nightly builds) and removed ready for review Issue is fixed, the pull request is being reviewed labels Aug 8, 2023
@isglass isglass added this to the 5.5.1 milestone Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working forum Issues from forum OEM OEM customer premium resolved Fixed but not yet released (available in the nightly builds)
Projects
None yet
Development

No branches or pull requests

4 participants