Skip to content
This repository has been archived by the owner on Oct 12, 2024. It is now read-only.

Do not take into account days that open and close at the same time #101

Closed
wants to merge 1 commit into from

Conversation

smeric
Copy link

@smeric smeric commented Nov 6, 2020

Hi,

Waiting for #87 to be addressed, I would suggest this small addition to disable days with same open and close time. Anyway it could be seen as something logical because if you open and close at the same time then you don't stay opened enough to allow any pick-up ;)

And thanks very much for this plugin !

Séb.

All Submissions:

Changes proposed in this Pull Request:

Closes # .

How to test the changes in this Pull Request:

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully run tests with your changes locally?

Changelog entry

Enter a summary of all changes on this Pull Request. This will appear in the changelog if accepted.

Hi,

Waiting for WC-Local-Pickup#87 to be addressed, I would suggest this small addition to disable days with same open and close time. Anyway it could be seen as something logical because if you open and close at the same time then you don't stay opened enough to allow any pick-up ;)

And thanks very much for this plugin !

Séb.
@smeric
Copy link
Author

smeric commented Nov 6, 2020

Sorry but I don't know how to pass all those tests :(

I've added a comment on #87.

@smeric smeric closed this Nov 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant