Skip to content

Instantly share code, notes, and snippets.

@stevendkwtz

stevendkwtz/todos_notes.md Secret

Created April 5, 2024 15:48
Show Gist options
  • Save stevendkwtz/9b2758642f151219b8a3f3753eee9e9d to your computer and use it in GitHub Desktop.
Save stevendkwtz/9b2758642f151219b8a3f3753eee9e9d to your computer and use it in GitHub Desktop.
Download ZIP
todos notes
Raw
todos_notes.md

  • DocsSamplesForTaskFormats.test - all the reminder examples do not have a date time, and it is the same date as due date

  • it.failing - have a look at them ✅ 2024-04-05

  • .length > 10 - hacky implementation ✅ 2024-04-05

    • Note: This was about how I added a test in tests/TaskSerializer/TaskSerializer.test.ts. This still exists. The test is still it.failing. I think we discovered there's some issues with how this test is constructed. That might be out of scope of this PR, but what might be in scope is just removing my changes to this file completely. Need your guidance here.

  • Review Clare's comment in the issues page. we haven't this yet implementation details

  • [/] @clare - decouple api, other changes as well

  • Reminders doocumentation incomplete

  • Filtering - does it work?

  • Discuss how to use scripts. Out of scope of this project. You were going to demo how to use the scripts in the project for development :)

Tests - Perhaps for pairing?

  • reminderdatefield.test.ts needed!! But TDD important here
    • Add thorough testing of reminder date searching
    • need tests explicitly testing date time wrt before/after
  • Explore the idea of expanding all date objs to support date time. In scope or out of scope?
  • [ ]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment