Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@zealot128
Copy link
Contributor

@zealot128 zealot128 commented Oct 11, 2017

First iteration with:

  • project filter (implicit by .gtt.yml or via arg),
  • scope (my/all)
  • state (opened/closed)

bildschirmfoto 2017-10-11 um 11 24 44
bildschirmfoto 2017-10-11 um 11 30 13

closes #26

Potential missing things:

  • Merge Request listings
  • show project name with namespace in table (not returned by the issue as it seems, only project_id, could scrape the project_name from the web_url though)
  • show already booked time alongside issue in table
  • pagination, sorting, per_page

* First iteration with project filter, scope (my/all), state (opened/closed) toggle
@coveralls
Copy link

coveralls commented Oct 11, 2017

Coverage Status

Coverage remained the same at 78.344% when pulling fe39396 on zealot128-os:gtt-list into 06de05b on kriskbx:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 78.344% when pulling fe39396 on zealot128-os:gtt-list into 06de05b on kriskbx:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 78.344% when pulling fe39396 on zealot128-os:gtt-list into 06de05b on kriskbx:master.

@kriskbx
Copy link
Owner

kriskbx commented Oct 12, 2017

You're on fire. :D

Good first iteration. Really like it. Instead of using the get method to query the API, you can use the all method which paginates through all pages automatically (and in parallel).

Pagination is a good thing to talk about because I'm not using pagination for the output right now. I don't know about you, but my gtt log gets pretty big and slow. Maybe we can introduce pagination for the output in the list method and refactor the log method to use the same functionality. What do you think?

@planser planser mentioned this pull request May 4, 2018
@MatthiasEgli-Audit
Copy link

Anything needed to get this merged?

@demisx
Copy link

demisx commented Feb 6, 2020

Can this be reviewed and merged please? What's holding it? This is way too cool project to let go stale.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add list command

5 participants