NotePlan / NotePlan/plugins

np.Templating: date.add and subtract and business add API/docs need normalization

Offen
#401 2 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@mikeerickson arbeitet bereits daran.

Seit 01.2.2023.

Vorherrschende Sprache
JavaScript
Sterne
204
Forks
82
Ø Merge
22 Std. 27 Min.
Gemergte PRs (30 T.)
3

Beschreibung

See the discussion here

date.add documentation is wrong
but even if the docs were right, the API would not be consistent with businessadd

Documentation:
Screen Cap 2023-02-01 at 09 14 33@2x

Note how pivotDate is in a different location for the two groups below.
Actual code in DateModule.js

  add(userPivotDate = '', value = '', shorthand = 'days', format = '') {
  subtract(userPivotDate = '', value = '', shorthand = 'days', format = '') {

  businessAdd(numDays = 1, pivotDate = '', format = '') {
  businessSubtract(numDays = 1, pivotDate = '', format = '') {

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.