Adding Sunday as a working day in Appointment Scheduling UI

I’m going to go back to a suggestion I made months ago and point out that we should just update the code so that it isn’t tied to the “current day” as much. Here is a PR that implements this.

It’s a bad idea to update a module default value just because it’s making our QA process more annoying.

The alternative would be what @dkayiwa suggests and add logic to the test to ensure that the includeWeekends value is true.

1 Like