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

CLOUDSTACK-9636: The host alerts box should be named as hosts in Alerts. #1803

Merged
merged 1 commit into from
Dec 2, 2016

Conversation

nitin-maharana
Copy link
Contributor

@nitin-maharana nitin-maharana commented Nov 30, 2016

The host Alerts box shows hosts in Alerts. The name host Alerts is misleading,
it should be changed to hosts in alerts.

For rest of the languages, it should be modified accordingly.
As I am not familiar with other languages, contributors familiar with other languages can suggest the change. or Open a new PR.

@jburwell
Copy link
Contributor

@nitin-maharana could you please add screen shots of the change to the PR?

@nitin-maharana
Copy link
Contributor Author

image

@rohityadavcloud
Copy link
Member

LGTM, @nitin-maharana can you rebase and change PR's base branch to 4.9. This is a small and useful change.

@nitin-maharana nitin-maharana changed the base branch from master to 4.9 December 1, 2016 08:09
asfgit pushed a commit that referenced this pull request Dec 2, 2016
CLOUDSTACK-9636: The host alerts box should be named as hosts in Alerts.The host Alerts box shows hosts in Alerts. The name host Alerts is misleading,
it should be changed to hosts in alerts.

For rest of the languages, it should be modified accordingly.
As I am not familiar with other languages, contributors familiar with other languages can suggest the change. or Open a new PR.

* pr/1803:
  CLOUDSTACK-9636: The host alerts box should be named as hosts in Alerts.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@asfgit asfgit merged commit 0198abb into apache:4.9 Dec 2, 2016
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.

4 participants