{"action":"created","issue":{"url":"https://api.github.com/repos/theforeman/foreman/issues/4760","repository_url":"https://api.github.com/repos/theforeman/foreman","labels_url":"https://api.github.com/repos/theforeman/foreman/issues/4760/labels{/name}","comments_url":"https://api.github.com/repos/theforeman/foreman/issues/4760/comments","events_url":"https://api.github.com/repos/theforeman/foreman/issues/4760/events","html_url":"https://github.com/theforeman/foreman/pull/4760","id":251587689,"number":4760,"title":"Fixes #20657 - make proxy labels bold","user":{"login":"ares","id":109773,"avatar_url":"https://avatars0.githubusercontent.com/u/109773?v=4","gravatar_id":"","url":"https://api.github.com/users/ares","html_url":"https://github.com/ares","followers_url":"https://api.github.com/users/ares/followers","following_url":"https://api.github.com/users/ares/following{/other_user}","gists_url":"https://api.github.com/users/ares/gists{/gist_id}","starred_url":"https://api.github.com/users/ares/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ares/subscriptions","organizations_url":"https://api.github.com/users/ares/orgs","repos_url":"https://api.github.com/users/ares/repos","events_url":"https://api.github.com/users/ares/events{/privacy}","received_events_url":"https://api.github.com/users/ares/received_events","type":"User","site_admin":false},"labels":[{"id":79040689,"url":"https://api.github.com/repos/theforeman/foreman/labels/Needs%20testing","name":"Needs testing","color":"fbca04","default":false},{"id":79040085,"url":"https://api.github.com/repos/theforeman/foreman/labels/Not%20yet%20reviewed","name":"Not yet reviewed","color":"009800","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2017-08-21T08:40:06Z","updated_at":"2017-08-22T08:08:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/theforeman/foreman/pulls/4760","html_url":"https://github.com/theforeman/foreman/pull/4760","diff_url":"https://github.com/theforeman/foreman/pull/4760.diff","patch_url":"https://github.com/theforeman/foreman/pull/4760.patch"},"body":"before:\r\n![before](https://user-images.githubusercontent.com/109773/29510714-0609f9b6-865d-11e7-9c82-ac850927d8eb.png)\r\nafter:\r\n![after](https://user-images.githubusercontent.com/109773/29510715-060ae254-865d-11e7-9ca5-cd6d4eca1554.png)\r\n\r\nshould be coordinated with plugins PRs"},"comment":{"url":"https://api.github.com/repos/theforeman/foreman/issues/comments/323951626","html_url":"https://github.com/theforeman/foreman/pull/4760#issuecomment-323951626","issue_url":"https://api.github.com/repos/theforeman/foreman/issues/4760","id":323951626,"user":{"login":"ares","id":109773,"avatar_url":"https://avatars0.githubusercontent.com/u/109773?v=4","gravatar_id":"","url":"https://api.github.com/users/ares","html_url":"https://github.com/ares","followers_url":"https://api.github.com/users/ares/followers","following_url":"https://api.github.com/users/ares/following{/other_user}","gists_url":"https://api.github.com/users/ares/gists{/gist_id}","starred_url":"https://api.github.com/users/ares/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ares/subscriptions","organizations_url":"https://api.github.com/users/ares/orgs","repos_url":"https://api.github.com/users/ares/repos","events_url":"https://api.github.com/users/ares/events{/privacy}","received_events_url":"https://api.github.com/users/ares/received_events","type":"User","site_admin":false},"created_at":"2017-08-22T08:08:22Z","updated_at":"2017-08-22T08:08:22Z","body":"@Rohoover I did changes in separate PRs keep discussion on every change isolated. This PR only addresses labels being bold. So the main question for this PR - ACK for bold labels?\r\n\r\nFor other items you suggested please see following PRs\r\n\r\n> Can we change that reload/refresh/revert icon to a button?\r\n\r\nhttps://github.com/theforeman/foreman/pull/4761\r\n\r\n> For \"Failed Features\" - What would go here? I don't think it should be left blank.\r\n\r\nalready fixed by https://github.com/theforeman/foreman/pull/4762\r\n\r\n> Can we change the labels to title case... \"Failed features\" = \"Failed Features\"\r\n\r\nI can do that, should I change all labels then? E.g. \"Configuration Status\", \"Active Features\", \"Hosts Managed\"? I would send a separate PR for this."}}
{"actor":{"display_login":"ares"}}