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

org-review-insert-last-review overrides NEXT_REVIEW #7

Open
bitozoid opened this issue Apr 9, 2022 · 1 comment
Open

org-review-insert-last-review overrides NEXT_REVIEW #7

bitozoid opened this issue Apr 9, 2022 · 1 comment

Comments

@bitozoid
Copy link

bitozoid commented Apr 9, 2022

This is an opinion about the behaviour of org-review-insert-last-review.

I would say that the property NEXT_REVIEW should not be overriden when inserting a last-review, unless the computed new next review comes after the current next review.

If the new computed next review comes before the current next review, it may mean that I reviewed the item before I had to, but that doesn't need to change the current review date. In this case, I want to register my last review but I do not want to change the current review date.

By the way, thanks for the software.

@brabalan
Copy link
Owner

I no longer use this, but I would gladly consider a pull request.

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