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

Fix modulus status command. #147

Merged
merged 1 commit into from
Sep 20, 2016
Merged

Fix modulus status command. #147

merged 1 commit into from
Sep 20, 2016

Conversation

theworkflow
Copy link
Contributor

Only print components part of the Modulus stack

Closes #144

if (err) return callback(err);
if (!body.components) return callback('Status not available');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might want to also check !body.components.length ?

Only print components part of the Modulus stack
Abort early if no components present
Copy link
Contributor

@jackboberg jackboberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

› modulus status
Welcome to Modulus
You are logged in as jack
--------------------------------------------------
Current Status
--------------------------------------------------
Web Interface:       operational
API:                 operational
MongoDB:             operational
Balancers:           operational
Stats:               operational
Application Hosts:   operational
Build Hosts:         operational
Support:             operational
--------------------------------------------------
View full details at https://status.progress.com
--------------------------------------------------

@theworkflow theworkflow merged commit 559a6e8 into master Sep 20, 2016
@theworkflow theworkflow deleted the bug/get-status branch September 20, 2016 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants