Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change behavior of /project list command #84

Closed
tannerwelsh opened this issue Aug 30, 2016 · 3 comments
Closed

Change behavior of /project list command #84

tannerwelsh opened this issue Aug 30, 2016 · 3 comments
Milestone

Comments

@tannerwelsh
Copy link
Contributor

tannerwelsh commented Aug 30, 2016

Based on LearnersGuild/echo#431, /project list -r will no longer be used.

The -r flag is to be removed, and /project list will display all active projects, the handles of their team, and the artifact url.

Example usage

@learner  > /project list
@echo     > There are 5 active projects.

            #terrible-tiger
            @billy, @bobby, @bud
            https://github.com/owner/repo

            #anxious-aardvark    
            @sally, @sarah, @sid
            https://github.com/owner/repo

            #silly-slug          
            @timmy, @tommy, @ted
            https://github.com/owner/repo

            #fantastic-falcon    
            @gabby, @gilly, @gaz
            https://github.com/owner/repo

            #creepy-caterpillar  
            @fanny, @fawn, @fred
            https://github.com/owner/repo

Updated help messages

For /project

/project - manage projects

Usage:
    /project [options] <command>

Commands:
    list         - List all active projects.

    set-artifact - Set the artifact for a project.



Options:
    --help, -h            print usage information


Examples:
    # List active projects, including team and artifact information
    /project list

    # Set the artifact for the #bad-lemurs-12 project.
    /project set-artifact #bad-lemurs-12 https://github.com/reactjs/redux

For /project list subcommand

/project list - show active projects along with their team and artifact

Usage:
    /project list [options]

Options:
    --help, -h            print usage information
@jeffreywescott
Copy link
Contributor

@tannerwelsh this looks great, although I think displaying team members inline like you suggest is going to cause very awkward formatting, since sometimes teams are as many as 5 people and handles are as long as 13-14 characters. It's going to make it really hard to read, and the columns won't line up.

@tannerwelsh
Copy link
Contributor Author

Good point @jeffreywescott. Just changed the output to make the project data multi-row instead of multi-column. Should reduce the total width of output, and make it more readable.

Omitting labels because the formatting of each line provides needed context to understand what it refers to (e.g. # for project name), and having a black line between projects creates an unambiguous separation.

@jeffreywescott jeffreywescott added this to the v2 (beta) milestone Sep 20, 2016
@jeffreywescott
Copy link
Contributor

Admin UI to the rescue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants