search

azendal / assembla_cli

an assembla.com cli
2 stars 0 forks source link

Vote to change method name my_tickets to my_report #13

Closed edgarjs closed 12 years ago

edgarjs commented 12 years ago

I was confused because I was expecting to get only my tickets, but this method returns actually custom reports. So I propose to change the name of the method to my_report and add an actual my_tickets method that returns only tickets assigned to me.

luisgodinez commented 12 years ago

Agree. If we want the list for our tickets only, we should use the tickets url, but with a different id for the report param, currently is 9, in deli space case, the tickets/report/9 return my closed tickets, 8 my active tickets, 7 all my tickets... etc.

And i think, tickets should return all the tickets, not only my tickets.

edgarjs commented 12 years ago

Actually I changed it to 3, because that was the right param

edgar.js

On Monday, January 9, 2012 at 5:29 PM, Luis Godinez wrote:

Agree. If we want the list for our tickets only, we should use the tickets url, but with a different id for the report param, currently is 9, in deli space case, the tickets/report/9 return my closed tickets, 8 my active tickets, 7 all my tickets... etc.

And i think, tickets should return all the tickets, not only my tickets.

Reply to this email directly or view it on GitHub: https://github.com/azendal/assembla_cli/issues/13#issuecomment-3422398

azendal commented 12 years ago

if you get all the tickets you are limited to 1,000 tickets and we have more than 3,000 tickets, i agree on the my_report, but my_tickets should return my active tickets, and tickets all tickets for the space (or at least the first 1000) and add the option to the page.

edgarjs commented 12 years ago

Right that's how it would be. In both cases it would be only active tickets. Maybe we need to add a closed_tickets method if you want to see the closed as well, I don't see any need for that though

edgar.js

On Monday, January 9, 2012 at 9:13 PM, Fernando Trasviña wrote:

if you get all the tickets you are limited to 1,000 tickets and we have more than 3,000 tickets, i agree on the my_report, but my_tickets should return my active tickets, and tickets all tickets for the space (or at least the first 1000) and add the option to the page.

Reply to this email directly or view it on GitHub: https://github.com/azendal/assembla_cli/issues/13#issuecomment-3424615