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

Remove warning about Reminders not working for recurring tasks, as the Reminders issue was resolved #2689

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adamfeldman
Copy link

@adamfeldman adamfeldman commented Mar 5, 2024

Description

Docs update. The reason this warning was documented was this: uphy/obsidian-reminder#93

Motivation and Context

How has this been tested?

Screenshots (if appropriate)

Types of changes

Changes visible to users:

  • Bug fix (prefix: fix - non-breaking change which fixes an issue)
  • New feature (prefix: feat - non-breaking change which adds functionality)
  • Breaking change (prefix: feat!! or fix!! - fix or feature that would cause existing functionality to not work as expected)
  • Documentation (prefix: docs - improvements to any documentation content for users)
  • Sample vault (prefix: vault - improvements to the Tasks-Demo sample vault)
  • Contributing Guidelines (prefix: contrib - any improvements to documentation content for contributors - see Contributing to Tasks)

Internal changes:

  • Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
  • Tests (prefix: test - additions and improvements to unit tests and the smoke tests)
  • Infrastructure (prefix: chore - examples include GitHub Actions, issue templates)

Checklist

Terms

@claremacrae
Copy link
Collaborator

Hi, thank you for the contribution.

I wonder whether you missed these checkboxes in the template accidentally:

image

This same change was offered in an earlier pull request - #2514 - in which I explained why just removing the warning was not sufficient.

If you would like to read the notes on that one, and address the concerns there, it would be greatly appreciated.

@claremacrae claremacrae added scope: documentation Improvements or additions to documentation question Further information is requested scope: recurrence Anything to do with recurring/repeating tasks labels Mar 5, 2024
@adamfeldman
Copy link
Author

HI @claremacrae, thanks for the pointer to #2514. I will take a look.

@adamfeldman adamfeldman marked this pull request as draft March 5, 2024 14:50
@claremacrae
Copy link
Collaborator

HI @claremacrae, thanks for the pointer to #2514. I will take a look.

Thanks Adam - and thanks too for updating the checklists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested scope: documentation Improvements or additions to documentation scope: recurrence Anything to do with recurring/repeating tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants