new/edit description should be optional
Open
Nobody has claimed this yet.
type: enhancement
- Dominant language
- Python
- Stars
- 3.1k
- Forks
- 233
- Avg merge
- 1h 44m
- Merged PRs (30d)
- 1
Description
(venv) allan@vent:~/src/khal$ khal new -i
summary: test
datetime range:
datetime range: now
timezone [EST5EDT]:
description (or "None"):
description (or "None"):
I.e. the first description should be accepted as valid input. I have the trivial fix ready to go once #1052 is merged.
Btw, if there is interest in #1050 then "accept default" and the "unset field" would become natural than the magic "None" string as either accepting the (possible) edited value and if new value is '' then it means unset.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing the issue with khal new -i and inspect the interactive description prompt. The change is done when the first empty description input is accepted as valid; account for the stated dependency on #1052 before proceeding.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- cli
- Issue type
- Bug
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100