Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Removed all of the code, tests, and templates from the old inspector #36

Merged
merged 2 commits into from Dec 18, 2013
Merged

Removed all of the code, tests, and templates from the old inspector #36

merged 2 commits into from Dec 18, 2013

Conversation

hatched
Copy link
Contributor

@hatched hatched commented Dec 18, 2013

No description provided.

@jujugui
Copy link
Contributor

jujugui commented Dec 18, 2013

Test PASSed.
Refer to this link for build results: http://ci.jujugui.org:8080/job/juju-gui/147/

@garyposter
Copy link
Contributor

👍 (and QA good)

As I said on IRC, I think you can remove line 1398 ("'juju-view-service',") from app/views/inspector.js

Thank you!

@hatched
Copy link
Contributor Author

hatched commented Dec 18, 2013

Thanks a lot for the review!

@jujugui
Copy link
Contributor

jujugui commented Dec 18, 2013

Test PASSed.
Refer to this link for build results: http://ci.jujugui.org:8080/job/juju-gui/149/

@hatched
Copy link
Contributor Author

hatched commented Dec 18, 2013

:shipit:

@jujugui
Copy link
Contributor

jujugui commented Dec 18, 2013

Status: merge request accepted. Url: http://ci.jujugui.org:8080/job/juju-gui-merge

jujugui added a commit that referenced this pull request Dec 18, 2013
@jujugui
Merge pull request #36 from hatched/remove-old-inspector-code
0ac2409
@jujugui jujugui merged commit 0ac2409 into juju:develop Dec 18, 2013
@hatched hatched deleted the remove-old-inspector-code branch December 19, 2013 00:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@hatched @jujugui @garyposter