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

Validation of Monthly byDayOfMonth should ensure day is selected #1584

Open
BobEvans opened this issue Oct 27, 2017 · 1 comment
Open

Validation of Monthly byDayOfMonth should ensure day is selected #1584

BobEvans opened this issue Oct 27, 2017 · 1 comment

Comments

@BobEvans
Copy link
Collaborator

No description provided.

@josmas
Copy link

josmas commented Mar 11, 2018

Is this for the web UI, scheduled triggers? If so, there are a few more things that need validation:

  • Weekly
    • Choose at least one day of the week to repeat on
  • Monthly
    • Either choose at least one day of the month
    • Or choose at least one day to repeat on (or is that Day of the week? see below)
  • ESM
    • Start/end times are selected (they can be deleted at the moment)
    • Start time is before end date
    • Minutes between is not negative

How does the Monthly --> Day of week work? Options are 1 to 5; are those weekdays? I can choose First and then repeat on S, M, T... what would be the trigger there?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants