Add show subtask#9
Conversation
|
@AlJohri This one is the more complete PR and is ready for review. I'm using it to dump tasks from asana and put them is an Elasticsearch. My process look like this : |
|
@theredcat nice stuff! unfortunately this project doesn't have any tests and I don't have an asana account so I can't test this, but the pull request looks small and reviewing the code it looks correct to me I'm happy to opt for the show tasks with subtasks option-- let's keep #9 and close #8? Also can you do a minor version bump to 0.0.4? after that, LGTM! |
|
also consider updating the help text in the README's CLI section to reflect the new command |
624f6aa to
50a2ec2
Compare
|
Yup this PR make #8 Obselete. I've orignally separated them to give you more options. I've updated the doc, bumped the version and rebased my commits to keep them self-sufficients |
|
Hi @AlJohri Could you take a look ? |
Follow Up for #8 to add subtask print as well.
I've splitted thoses PR if you want to keep the simple show command without the subtask option.