NotePlan / NotePlan/plugins

calendarNoteByDateString() not working for uninitialised notes

Open
#424 0 comments 0 reactions 1 assignee View on GitHub

@EduardMe is already working on this.

Since Mar 15, 2023.

API bug
Dominant language
JavaScript
Stars
204
Forks
82
Avg merge
22h 27m
Merged PRs (30d)
3

Description

Not strictly a bug, but its related to an earlier bug that did get fixed.

Back around v3.1 asking calendarNoteByDate() for a future but as-yet uninitialised calendar note resulted in null. You fixed this behaviour.

Since then you've added calendarNoteByDateString() but I've only just noticed that its documentation says:

"Returns the calendar note for the given date string (can be undefined, if the calendar note was not created yet) ...".

Please update this call to act the same as its older brother.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.