oesmith / ralio

A usable command-line client for Rally
https://github.com/oesmith/ralio
30 stars 21 forks source link

Discussion reading and commenting support #38

Closed Two9A closed 11 years ago

Two9A commented 11 years ago

Introduces the discuss command, which can be used in one of two ways:

ralio discuss DE123 will list the discussion posts on the artifact, in order; ralio discuss --add "Should be fixed." TA461 will add a comment onto the discussion, and list the new discussion in full.

igorescobar commented 11 years ago

There is anyway that we could delete and edit those discussion posts?

Would be nice if we could use it just like the ralio task command with the add and delete command and not --add and --delete.

Two9A commented 11 years ago

Discussion appears to be for quick notes against a ticket. As far as I can tell, discussion comments can't be edited or removed once they've been left; there's no interface in the Web UI for it, at least.

igorescobar commented 11 years ago

Cool, no problem. What about the ralio discussion add instead of --add? Just to keep the same API as ralio task command.

Thanks for your contributions @Two9A really useful!

igorescobar commented 11 years ago

My mistake, the command is ralio task create instead of add.

igorescobar commented 11 years ago

@Two9A In that case, maybe is more appropriate to use "add" instead of "create". Can you make the adjustment to the command work with add anstead of --add ?

igorescobar commented 11 years ago

What about loading discussions on the command ralio show? Do you think that would be nice? :+1: I could help you to implement that if you want :-)

igorescobar commented 11 years ago

Im going to merge this pullrequest and then we can discuss it in another episode :)

igorescobar commented 11 years ago

Thank you @Two9A , you rock! :+1:

igorescobar commented 11 years ago

I did small changes and a small bugfix on your pull-request but its all done in v0.6.0 of ralio, already available at npm repo.

Two9A commented 11 years ago

That's great, thanks. As for having discussions show up with ralio show, it's a good idea; perhaps as an additional switch.

Maybe the number of discussion comments could come through on ralio sprint as well? Things to think about...

igorescobar commented 11 years ago

I think we could do it on baby steps. At the beginning we could only show the discussion thread on ralio show command, then we could start developing more features around the discussion feature and add more visibility of this data. As long as I can tell I never seen anybody using that "discussion" feature at Rally Web. Maybe we can keep that new ideas on hold and wait for an more concrete data of what the user really wants.