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.

Sign up for GitHub

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

Jump to bottom

Add site:list command #17

Closed
tomjanssens opened this issue Apr 15, 2016 · 2 comments · Fixed by #54
Closed

Add site:list command #17

tomjanssens opened this issue Apr 15, 2016 · 2 comments · Fixed by #54
Milestone
1.4.7

Comments

@tomjanssens
Copy link

List all available sites.
@tomjanssens tomjanssens added this to the Current milestone Apr 15, 2016
@rodolfovillaruz rodolfovillaruz mentioned this issue Aug 31, 2016
Closed
rodolfovillaruz added a commit that referenced this issue Sep 1, 2016
@rodolfovillaruz
#17 - Remove displayed comment
2972ef1
rodolfovillaruz added a commit that referenced this issue Sep 1, 2016
@rodolfovillaruz
#17 - Full-hand notation on foreach control structure
c1f6d3f
@stevenrombauts
Copy link
Member

@rodolfovillaruz An additional improvement to this ticket would be to add an --format option. We could have two formats: text and json.

Since we are now re-using the same code from the joomlatools-vagrant dashboard file, we could replace that code with a call to site:list to get the json data and then parse it out. If we add new versions or fixes we only have to update the code once, in joomlatools-console.

rodolfovillaruz added a commit that referenced this issue Sep 1, 2016
@rodolfovillaruz
#17: List site class
c09a8ac
rodolfovillaruz added a commit that referenced this issue Sep 1, 2016
@rodolfovillaruz
#17: Remove displayed comment
7dc66de
rodolfovillaruz added a commit that referenced this issue Sep 1, 2016
@rodolfovillaruz
#17: Full-hand notation on foreach control structure
ca57a4e
rodolfovillaruz added a commit that referenced this issue Sep 23, 2016
@rodolfovillaruz
#17: JSON formatting functionality
721eedd
@johanjanssens
Copy link
Member

@stevenrombauts Can you please review this?

@stevenrombauts stevenrombauts mentioned this issue Sep 26, 2016
Closed
rodolfovillaruz added a commit that referenced this issue Oct 7, 2016
@rodolfovillaruz
#17: Compatibility issue and default control statement
296ed81
stevenrombauts pushed a commit that referenced this issue Oct 10, 2016
#17: Improve output
6086564
@rodolfovillaruz rodolfovillaruz mentioned this issue Oct 10, 2016
Merged
@stevenrombauts stevenrombauts modified the milestones: 1.4.7, 1.5.0 Jan 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.4.7
Development

Successfully merging a pull request may close this issue.

Add site:list command joomlatools/joomlatools-console
4 participants
@stevenrombauts @johanjanssens @tomjanssens @rodolfovillaruz