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

cmd/grit: Additions to various commands for quality of life #32

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

anhnamtran
Copy link

@anhnamtran anhnamtran commented Apr 26, 2021

Closes #28
Closes #9

This pull request adds the following changes:

  • grit add [-a|--alias] ALIAS ...: alias a node on addition.
  • grit list|ls all: list all nodes
  • grit move|mv NODE ORIGIN TARGET: Move a NODE from ORIGIN to TARGET

This commit allows the option `-a|--alias` for `grit add` that will
allow aliasing a node on addition.
`grit list` will now recognize "all" as a special selector, listing
*all* nodes including children nodes.
Add new "move" command that allows a node to be moved from one parent to
another.

It simply aliases the action of unlink a node and re-linking it with a
different parent.
The options allow unlinking a node from all children or all parents,
these options are mutually exclusive (including the existing "target"
arguemnts)
The app crashes if no node for the current day has been created, either
from `add` or something else. This allows the user to specify `today` as
a keyword in order to automatically add the date node.
`grit tree -u` will sort the nodes by order of Inactive -> InProgress ->
Completed. This is helpful in seeing what needs to be done still.
@vaygr
Copy link

vaygr commented Aug 1, 2022

These seem like very useful fixes. @climech do you think it's something that can be merged soon?

@onordesjo
Copy link

Agreed, this would be very useful, especially 👍 for moving nodes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

List all nodes Moving nodes
3 participants