improved list output #281

Merged
6543 merged 12 commits from noerw/tea:issue-244-sorted-list into master 2 years ago
noerw commented 2 years ago
Collaborator

Refactors list output, enabling common functionality on all listings:

  • For now, only sorting is implemented
  • We could also could add dynamic field selection like currently in modules/print/repo.go, though that would require much more refactoring I think. Any ideas? This will be part of another PR if we really need it

fixes #244

Refactors list output, enabling common functionality on all listings: - For now, only sorting is implemented - ~~We could also could add dynamic field selection like currently in `modules/print/repo.go`, though that would require much more refactoring I think. *Any ideas?*~~ This will be part of another PR if we really need it fixes #244
noerw added 5 commits 2 years ago
noerw added the
kind/feature
kind/refactor
status/needs-feedback
labels 2 years ago
noerw added 1 commit 2 years ago
b4dfb9a28f
sort milestones descending
noerw added 2 commits 2 years ago
Collaborator

I think we should use dynamik field selection, but only on specific lists

The issue list for example shoud have it, ... this way we can unify IssuesList, IssuesPullsList and PullsList into a single one ...

I think we should use dynamik field selection, but only on specific lists The issue list for example shoud have it, ... this way we can unify IssuesList, IssuesPullsList and PullsList into a single one ...
noerw added
status/needs-reviews
and removed
status/needs-feedback
labels 2 years ago
noerw added 1 commit 2 years ago
027ac7c6ea Merge branch 'master' into issue-244-sorted-list
noerw added 1 commit 2 years ago
f4bc0684dd Merge branch 'master' into issue-244-sorted-list
khmarbaise approved these changes 2 years ago
Dismissed
6543 added 1 commit 2 years ago
070a75f3e8 Merge branch 'master' into issue-244-sorted-list
6543 added 1 commit 2 years ago
a65de98854 rm empty line
6543 approved these changes 2 years ago
Dismissed
6543 removed the
status/needs-reviews
label 2 years ago
6543 merged commit a91168fd36 into master 2 years ago
6543 referenced this issue from a commit 2 years ago
6543 added this to the v0.6.0 milestone 2 years ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as a91168fd36.
Sign in to join this conversation.
Loading…
There is no content yet.