{"action":"created","issue":{"url":"https://api.github.com/repos/ezsystems/hybrid-platform-ui/issues/30","repository_url":"https://api.github.com/repos/ezsystems/hybrid-platform-ui","labels_url":"https://api.github.com/repos/ezsystems/hybrid-platform-ui/issues/30/labels{/name}","comments_url":"https://api.github.com/repos/ezsystems/hybrid-platform-ui/issues/30/comments","events_url":"https://api.github.com/repos/ezsystems/hybrid-platform-ui/issues/30/events","html_url":"https://github.com/ezsystems/hybrid-platform-ui/pull/30","id":237867787,"number":30,"title":"[WIP] EZP-27543: Integrate PJAX notification in Hybrid Platform UI App","user":{"login":"dpobel","id":305563,"avatar_url":"https://avatars1.githubusercontent.com/u/305563?v=3","gravatar_id":"","url":"https://api.github.com/users/dpobel","html_url":"https://github.com/dpobel","followers_url":"https://api.github.com/users/dpobel/followers","following_url":"https://api.github.com/users/dpobel/following{/other_user}","gists_url":"https://api.github.com/users/dpobel/gists{/gist_id}","starred_url":"https://api.github.com/users/dpobel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dpobel/subscriptions","organizations_url":"https://api.github.com/users/dpobel/orgs","repos_url":"https://api.github.com/users/dpobel/repos","events_url":"https://api.github.com/users/dpobel/events{/privacy}","received_events_url":"https://api.github.com/users/dpobel/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-06-22T14:37:34Z","updated_at":"2017-06-22T14:49:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ezsystems/hybrid-platform-ui/pulls/30","html_url":"https://github.com/ezsystems/hybrid-platform-ui/pull/30","diff_url":"https://github.com/ezsystems/hybrid-platform-ui/pull/30.diff","patch_url":"https://github.com/ezsystems/hybrid-platform-ui/pull/30.patch"},"body":"JIRA: https://jira.ez.no/browse/EZP-27543\r\nRequires: https://github.com/ezsystems/PlatformUIBundle/pull/881\r\n\r\n# Description\r\n\r\n**WIP** Attempt to have PJAX notification visible in the app\r\n\r\n## Tasks\r\n\r\n* [x] Change server side App component to be have notifications\r\n* [x] Parse PJAX notifications\r\n* [ ] Figure out how to test that with php spec (@bdunogier I'm gonna need your help)"},"comment":{"url":"https://api.github.com/repos/ezsystems/hybrid-platform-ui/issues/comments/310403396","html_url":"https://github.com/ezsystems/hybrid-platform-ui/pull/30#issuecomment-310403396","issue_url":"https://api.github.com/repos/ezsystems/hybrid-platform-ui/issues/30","id":310403396,"user":{"login":"ezrobot","id":681611,"avatar_url":"https://avatars2.githubusercontent.com/u/681611?v=3","gravatar_id":"","url":"https://api.github.com/users/ezrobot","html_url":"https://github.com/ezrobot","followers_url":"https://api.github.com/users/ezrobot/followers","following_url":"https://api.github.com/users/ezrobot/following{/other_user}","gists_url":"https://api.github.com/users/ezrobot/gists{/gist_id}","starred_url":"https://api.github.com/users/ezrobot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ezrobot/subscriptions","organizations_url":"https://api.github.com/users/ezrobot/orgs","repos_url":"https://api.github.com/users/ezrobot/repos","events_url":"https://api.github.com/users/ezrobot/events{/privacy}","received_events_url":"https://api.github.com/users/ezrobot/received_events","type":"User","site_admin":false},"created_at":"2017-06-22T14:49:42Z","updated_at":"2017-06-22T14:49:42Z","body":"Tool version : PHP CS Fixer 2.1.2 by Fabien Potencier and Dariusz Ruminski\nCommand executed ```php-cs-fixer --dry-run -v fix ```\nThis Pull Request does not respect [PSR-2 Coding Standards](http://www.php-fig.org/psr/psr-2/), please, see the suggested diff below:\n\n```diff\ndiff --git a/src/lib/Pjax/PjaxResponseMainContentMapper.php b/src/lib/Pjax/PjaxResponseMainContentMapper.php\nindex ec3894e..c717500 100644\n--- a/src/lib/Pjax/PjaxResponseMainContentMapper.php\n+++ b/src/lib/Pjax/PjaxResponseMainContentMapper.php\n@@ -98,6 +98,7 @@ class PjaxResponseMainContentMapper implements MainContentMapper\n                 'content' => $this->innerHtml($element),\n             ];\n         }\n+\n         return $notifications;\n     }\n \n```\n"}}
{"actor":{"display_login":"ezrobot"}}