{"action":"created","issue":{"url":"https://api.github.com/repos/apiman/apiman/issues/336","labels_url":"https://api.github.com/repos/apiman/apiman/issues/336/labels{/name}","comments_url":"https://api.github.com/repos/apiman/apiman/issues/336/comments","events_url":"https://api.github.com/repos/apiman/apiman/issues/336/events","html_url":"https://github.com/apiman/apiman/pull/336","id":128015868,"number":336,"title":"Ignore Resources Policy: enhance to allow shading by VERB","user":{"login":"wtrocki","id":981838,"avatar_url":"https://avatars.githubusercontent.com/u/981838?v=3","gravatar_id":"","url":"https://api.github.com/users/wtrocki","html_url":"https://github.com/wtrocki","followers_url":"https://api.github.com/users/wtrocki/followers","following_url":"https://api.github.com/users/wtrocki/following{/other_user}","gists_url":"https://api.github.com/users/wtrocki/gists{/gist_id}","starred_url":"https://api.github.com/users/wtrocki/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wtrocki/subscriptions","organizations_url":"https://api.github.com/users/wtrocki/orgs","repos_url":"https://api.github.com/users/wtrocki/repos","events_url":"https://api.github.com/users/wtrocki/events{/privacy}","received_events_url":"https://api.github.com/users/wtrocki/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-01-21T20:46:31Z","updated_at":"2016-01-22T10:11:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apiman/apiman/pulls/336","html_url":"https://github.com/apiman/apiman/pull/336","diff_url":"https://github.com/apiman/apiman/pull/336.diff","patch_url":"https://github.com/apiman/apiman/pull/336.patch"},"body":"## What was done\r\n- Added verb (http method) support for Ignore resources policy rule.\r\n- Adjusted tests to cover new cases. \r\n- Reshaped UI to support new functionalities. \r\n\r\n## Gui changes\r\n\r\n`IgnoredAligators`\r\nhttp://gfycat.com/ThatBowedAlligatorgar\r\n\r\n`AuthBorer`\r\nhttp://gfycat.com/AliveKindlyBorer\r\n\r\n## Technical comments\r\n\r\n1. Policies gui components\r\nI think we may improve a little bit the way we style policies. \r\nCurrently we use nested style (which is slightly invalid) or direct style on elements. \r\nI think that we can have separate style templates for this (with gulp support). \r\nIt might look better to clean all rules at the same time so I'm not pushing that here.\r\n\r\n2. Separate ts files per policy.\r\nIt may be better to have separate ts files per policy (this may speed up gulp building)\r\nDo we have any reasons to keep it in one file?"},"comment":{"url":"https://api.github.com/repos/apiman/apiman/issues/comments/173868519","html_url":"https://github.com/apiman/apiman/pull/336#issuecomment-173868519","issue_url":"https://api.github.com/repos/apiman/apiman/issues/336","id":173868519,"user":{"login":"wtrocki","id":981838,"avatar_url":"https://avatars.githubusercontent.com/u/981838?v=3","gravatar_id":"","url":"https://api.github.com/users/wtrocki","html_url":"https://github.com/wtrocki","followers_url":"https://api.github.com/users/wtrocki/followers","following_url":"https://api.github.com/users/wtrocki/following{/other_user}","gists_url":"https://api.github.com/users/wtrocki/gists{/gist_id}","starred_url":"https://api.github.com/users/wtrocki/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wtrocki/subscriptions","organizations_url":"https://api.github.com/users/wtrocki/orgs","repos_url":"https://api.github.com/users/wtrocki/repos","events_url":"https://api.github.com/users/wtrocki/events{/privacy}","received_events_url":"https://api.github.com/users/wtrocki/received_events","type":"User","site_admin":false},"created_at":"2016-01-22T10:11:08Z","updated_at":"2016-01-22T10:11:08Z","body":"Tests are passing locally. Investigating that at the moment."}}