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

fix repeater adjusting time of date #746

Merged
merged 4 commits into from
Nov 9, 2021

Conversation

tarnung
Copy link
Collaborator

@tarnung tarnung commented Nov 7, 2021

Should fix #683

Since I'm still not using emacs, someone might want to check if my assumptions based on org mode docs are correct.

using ".+":

  • when repeaterUnit is not "h", hour and minutes are never touched.
  • when repeaterUnit is "h", hour and minutes are set to n hour(s) from now (as it was already implemented before this fix)

@munen
Copy link
Collaborator

munen commented Nov 9, 2021

Hi @tarnung

Good job on this PR! 🙏

I confirm your statements on how the repeaters work. I have also added tests to showcase it. Unfortunately, adding these new tests makes an old test fail and I do not understand why. I have spent quite a bit of time trying to figure out what the problem is, but it shouldn't block merging this PR as it is unrelated. I'll open a separate PR for it.

Great job, thank you 🙏

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.

When updating a reapeated task with timestamp, do not update the time portion
2 participants