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

Status Fixes and Error Tracking #764

Merged
merged 3 commits into from
Nov 9, 2015
Merged

Status Fixes and Error Tracking #764

merged 3 commits into from
Nov 9, 2015

Conversation

brittainhard
Copy link
Contributor

Closes #762

@ahmadia
Copy link
Contributor

ahmadia commented Nov 9, 2015

Sounds good and looks good. I think the status gets "rendered" but the dashboard doesn't recognize it and can't respond.

@brittainhard
Copy link
Contributor Author

I'll switch it to recognize. That makes much more sense.

@ahmadia
Copy link
Contributor

ahmadia commented Nov 9, 2015

+1 to merge when you're ready.

brittainhard added a commit that referenced this pull request Nov 9, 2015
@brittainhard brittainhard merged commit 6d7baea into master Nov 9, 2015
@brittainhard brittainhard deleted the bhard/status_errors branch November 9, 2015 17:27
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

Successfully merging this pull request may close these issues.

2 participants