Skip to content

feat(nlp): better every day text for weekly frequency#643

Open
eemelipa wants to merge 1 commit intojkbrzt:masterfrom
eemelipa:534-fix-wording
Open

feat(nlp): better every day text for weekly frequency#643
eemelipa wants to merge 1 commit intojkbrzt:masterfrom
eemelipa:534-fix-wording

Conversation

@eemelipa
Copy link
Copy Markdown

@eemelipa eemelipa commented Nov 1, 2024

Fixes #534


Thanks for contributing to rrule!

To submit a pull request, please verify that you have done the following:

  • Merged in or rebased on the latest master commit
  • Linked to an existing bug or issue describing the bug or feature you're
    addressing
  • Written one or more tests showing that your change works as advertised

'Every 2 weeks on every day',
'RRULE:INTERVAL=2;FREQ=WEEKLY;BYDAY=MO,TU,WE,TH,FR,SA,SU',
],
['Every 2 weeks on Monday', 'RRULE:INTERVAL=2;FREQ=WEEKLY;BYDAY=MO'],
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verifies mostly that existing "Every X weeks on Y" didn't get broken

@eemelipa eemelipa marked this pull request as ready for review November 1, 2024 17:55
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

Successfully merging this pull request may close these issues.

Weird rule text when select the whole week and set the interval

1 participant