{"action":"created","issue":{"url":"https://api.github.com/repos/civicrm/civihr/issues/1065","repository_url":"https://api.github.com/repos/civicrm/civihr","labels_url":"https://api.github.com/repos/civicrm/civihr/issues/1065/labels{/name}","comments_url":"https://api.github.com/repos/civicrm/civihr/issues/1065/comments","events_url":"https://api.github.com/repos/civicrm/civihr/issues/1065/events","html_url":"https://github.com/civicrm/civihr/pull/1065","id":169922967,"number":1065,"title":"UY-51 - Fixing of fetching Job Contract entities (pay and health).","user":{"login":"coldrunKacper","id":8986209,"avatar_url":"https://avatars.githubusercontent.com/u/8986209?v=3","gravatar_id":"","url":"https://api.github.com/users/coldrunKacper","html_url":"https://github.com/coldrunKacper","followers_url":"https://api.github.com/users/coldrunKacper/followers","following_url":"https://api.github.com/users/coldrunKacper/following{/other_user}","gists_url":"https://api.github.com/users/coldrunKacper/gists{/gist_id}","starred_url":"https://api.github.com/users/coldrunKacper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coldrunKacper/subscriptions","organizations_url":"https://api.github.com/users/coldrunKacper/orgs","repos_url":"https://api.github.com/users/coldrunKacper/repos","events_url":"https://api.github.com/users/coldrunKacper/events{/privacy}","received_events_url":"https://api.github.com/users/coldrunKacper/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-08-08T13:28:51Z","updated_at":"2016-08-08T13:59:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/civicrm/civihr/pulls/1065","html_url":"https://github.com/civicrm/civihr/pull/1065","diff_url":"https://github.com/civicrm/civihr/pull/1065.diff","patch_url":"https://github.com/civicrm/civihr/pull/1065.patch"},"body":"The issue was caused by these two commits in the past:\r\nhttps://github.com/civicrm/civihr/commit/f5da231fc6486558f97e98e4af6a15fb3937bfe7\r\nand\r\nhttps://github.com/civicrm/civihr/commit/4210a556cd231ffe5ee0dc9f511cb2492a0bae0d\r\nwhich seem to move Pay and Health entity fetching below the rest of the entities. The issue was that for both entities there was used Details revision ID instead of particular entities revision.\r\nI've moved Pay and Health fetching into the $q.all() method to get current valid revisions."},"comment":{"url":"https://api.github.com/repos/civicrm/civihr/issues/comments/238246657","html_url":"https://github.com/civicrm/civihr/pull/1065#issuecomment-238246657","issue_url":"https://api.github.com/repos/civicrm/civihr/issues/1065","id":238246657,"user":{"login":"danilo-compucorp","id":18520391,"avatar_url":"https://avatars.githubusercontent.com/u/18520391?v=3","gravatar_id":"","url":"https://api.github.com/users/danilo-compucorp","html_url":"https://github.com/danilo-compucorp","followers_url":"https://api.github.com/users/danilo-compucorp/followers","following_url":"https://api.github.com/users/danilo-compucorp/following{/other_user}","gists_url":"https://api.github.com/users/danilo-compucorp/gists{/gist_id}","starred_url":"https://api.github.com/users/danilo-compucorp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danilo-compucorp/subscriptions","organizations_url":"https://api.github.com/users/danilo-compucorp/orgs","repos_url":"https://api.github.com/users/danilo-compucorp/repos","events_url":"https://api.github.com/users/danilo-compucorp/events{/privacy}","received_events_url":"https://api.github.com/users/danilo-compucorp/received_events","type":"User","site_admin":false},"created_at":"2016-08-08T13:59:41Z","updated_at":"2016-08-08T13:59:41Z","body":"Approved"}}
{"actor":{"display_login":"danilo-compucorp"}}