{"action":"created","comment":{"url":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/comments/39501246","id":39501246,"diff_hunk":"@@ -18,13 +18,13 @@ function resultMessage(success, verb, conceptName) {\n }\n \n function fetchUserPrefs(type, userId, verb, subjectType) {\n-\tloadTestMyftApi.get(userId, verb);","path":"server/controllers/my-topics.js","position":15,"original_position":15,"commit_id":"cf1de0fd94433b69b0489094a21a998735965675","original_commit_id":"58ce089e64692ce28507d5554f1759313c6913ef","user":{"login":"phamann","id":80089,"avatar_url":"https://avatars.githubusercontent.com/u/80089?v=3","gravatar_id":"","url":"https://api.github.com/users/phamann","html_url":"https://github.com/phamann","followers_url":"https://api.github.com/users/phamann/followers","following_url":"https://api.github.com/users/phamann/following{/other_user}","gists_url":"https://api.github.com/users/phamann/gists{/gist_id}","starred_url":"https://api.github.com/users/phamann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phamann/subscriptions","organizations_url":"https://api.github.com/users/phamann/orgs","repos_url":"https://api.github.com/users/phamann/repos","events_url":"https://api.github.com/users/phamann/events{/privacy}","received_events_url":"https://api.github.com/users/phamann/received_events","type":"User","site_admin":false},"body":"I've now resolved this issue inside the proxy: https://github.com/Financial-Times/next-ilyushin/blob/use-api-client/server/lib/api-proxy.js#L53","created_at":"2015-09-15T11:55:18Z","updated_at":"2015-09-15T11:55:18Z","html_url":"https://github.com/Financial-Times/next-ilyushin/pull/301#discussion_r39501246","pull_request_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/301","_links":{"self":{"href":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/comments/39501246"},"html":{"href":"https://github.com/Financial-Times/next-ilyushin/pull/301#discussion_r39501246"},"pull_request":{"href":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/301"}}},"pull_request":{"url":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/301","id":44942510,"html_url":"https://github.com/Financial-Times/next-ilyushin/pull/301","diff_url":"https://github.com/Financial-Times/next-ilyushin/pull/301.diff","patch_url":"https://github.com/Financial-Times/next-ilyushin/pull/301.patch","issue_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/issues/301","number":301,"state":"open","locked":false,"title":"Use api client","user":{"login":"phamann","id":80089,"avatar_url":"https://avatars.githubusercontent.com/u/80089?v=3","gravatar_id":"","url":"https://api.github.com/users/phamann","html_url":"https://github.com/phamann","followers_url":"https://api.github.com/users/phamann/followers","following_url":"https://api.github.com/users/phamann/following{/other_user}","gists_url":"https://api.github.com/users/phamann/gists{/gist_id}","starred_url":"https://api.github.com/users/phamann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phamann/subscriptions","organizations_url":"https://api.github.com/users/phamann/orgs","repos_url":"https://api.github.com/users/phamann/repos","events_url":"https://api.github.com/users/phamann/events{/privacy}","received_events_url":"https://api.github.com/users/phamann/received_events","type":"User","site_admin":false},"body":"This is a big PR which is mainly @Lrnk's great handy work.\r\n\r\n- Wrap all calls to data APIs in the `myftApiV2` switch to toggle between source\r\n- When switch is true use the new isomorphic `myFtApi` client to request data\r\n- When switch is false map the old data structure to the new one\r\n- Update API proxy\r\n- Tests\r\n\r\n/cc @adgad @Lrnk @wheresrhys @keirog ","created_at":"2015-09-14T15:16:14Z","updated_at":"2015-09-15T11:55:18Z","closed_at":null,"merged_at":null,"merge_commit_sha":"01110a1f2b194e8ba177efcd5504eaf1100d6a09","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/301/commits","review_comments_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/301/comments","review_comment_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/issues/301/comments","statuses_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/statuses/cf1de0fd94433b69b0489094a21a998735965675","head":{"label":"Financial-Times:use-api-client","ref":"use-api-client","sha":"cf1de0fd94433b69b0489094a21a998735965675","user":{"login":"Financial-Times","id":3502508,"avatar_url":"https://avatars.githubusercontent.com/u/3502508?v=3","gravatar_id":"","url":"https://api.github.com/users/Financial-Times","html_url":"https://github.com/Financial-Times","followers_url":"https://api.github.com/users/Financial-Times/followers","following_url":"https://api.github.com/users/Financial-Times/following{/other_user}","gists_url":"https://api.github.com/users/Financial-Times/gists{/gist_id}","starred_url":"https://api.github.com/users/Financial-Times/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Financial-Times/subscriptions","organizations_url":"https://api.github.com/users/Financial-Times/orgs","repos_url":"https://api.github.com/users/Financial-Times/repos","events_url":"https://api.github.com/users/Financial-Times/events{/privacy}","received_events_url":"https://api.github.com/users/Financial-Times/received_events","type":"Organization","site_admin":false},"repo":{"id":28864313,"name":"next-ilyushin","full_name":"Financial-Times/next-ilyushin","owner":{"login":"Financial-Times","id":3502508,"avatar_url":"https://avatars.githubusercontent.com/u/3502508?v=3","gravatar_id":"","url":"https://api.github.com/users/Financial-Times","html_url":"https://github.com/Financial-Times","followers_url":"https://api.github.com/users/Financial-Times/followers","following_url":"https://api.github.com/users/Financial-Times/following{/other_user}","gists_url":"https://api.github.com/users/Financial-Times/gists{/gist_id}","starred_url":"https://api.github.com/users/Financial-Times/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Financial-Times/subscriptions","organizations_url":"https://api.github.com/users/Financial-Times/orgs","repos_url":"https://api.github.com/users/Financial-Times/repos","events_url":"https://api.github.com/users/Financial-Times/events{/privacy}","received_events_url":"https://api.github.com/users/Financial-Times/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Financial-Times/next-ilyushin","description":"My Page (e.g. following, recommended, saved for later)","fork":false,"url":"https://api.github.com/repos/Financial-Times/next-ilyushin","forks_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/forks","keys_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/teams","hooks_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/hooks","issue_events_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/issues/events{/number}","events_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/events","assignees_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/assignees{/user}","branches_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/branches{/branch}","tags_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/tags","blobs_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/git/refs{/sha}","trees_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/statuses/{sha}","languages_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/languages","stargazers_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/stargazers","contributors_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/contributors","subscribers_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/subscribers","subscription_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/subscription","commits_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/commits{/sha}","git_commits_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/git/commits{/sha}","comments_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/comments{/number}","issue_comment_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/issues/comments{/number}","contents_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/contents/{+path}","compare_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/merges","archive_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/downloads","issues_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/issues{/number}","pulls_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls{/number}","milestones_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/milestones{/number}","notifications_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/labels{/name}","releases_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/releases{/id}","created_at":"2015-01-06T13:18:01Z","updated_at":"2015-05-07T15:08:11Z","pushed_at":"2015-09-15T11:54:08Z","git_url":"git://github.com/Financial-Times/next-ilyushin.git","ssh_url":"git@github.com:Financial-Times/next-ilyushin.git","clone_url":"https://github.com/Financial-Times/next-ilyushin.git","svn_url":"https://github.com/Financial-Times/next-ilyushin","homepage":null,"size":26631,"stargazers_count":0,"watchers_count":0,"language":"JavaScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":12,"forks":0,"open_issues":12,"watchers":0,"default_branch":"master"}},"base":{"label":"Financial-Times:master","ref":"master","sha":"12ac4f150245251ed8a52d95e26022469a8be844","user":{"login":"Financial-Times","id":3502508,"avatar_url":"https://avatars.githubusercontent.com/u/3502508?v=3","gravatar_id":"","url":"https://api.github.com/users/Financial-Times","html_url":"https://github.com/Financial-Times","followers_url":"https://api.github.com/users/Financial-Times/followers","following_url":"https://api.github.com/users/Financial-Times/following{/other_user}","gists_url":"https://api.github.com/users/Financial-Times/gists{/gist_id}","starred_url":"https://api.github.com/users/Financial-Times/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Financial-Times/subscriptions","organizations_url":"https://api.github.com/users/Financial-Times/orgs","repos_url":"https://api.github.com/users/Financial-Times/repos","events_url":"https://api.github.com/users/Financial-Times/events{/privacy}","received_events_url":"https://api.github.com/users/Financial-Times/received_events","type":"Organization","site_admin":false},"repo":{"id":28864313,"name":"next-ilyushin","full_name":"Financial-Times/next-ilyushin","owner":{"login":"Financial-Times","id":3502508,"avatar_url":"https://avatars.githubusercontent.com/u/3502508?v=3","gravatar_id":"","url":"https://api.github.com/users/Financial-Times","html_url":"https://github.com/Financial-Times","followers_url":"https://api.github.com/users/Financial-Times/followers","following_url":"https://api.github.com/users/Financial-Times/following{/other_user}","gists_url":"https://api.github.com/users/Financial-Times/gists{/gist_id}","starred_url":"https://api.github.com/users/Financial-Times/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Financial-Times/subscriptions","organizations_url":"https://api.github.com/users/Financial-Times/orgs","repos_url":"https://api.github.com/users/Financial-Times/repos","events_url":"https://api.github.com/users/Financial-Times/events{/privacy}","received_events_url":"https://api.github.com/users/Financial-Times/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Financial-Times/next-ilyushin","description":"My Page (e.g. following, recommended, saved for later)","fork":false,"url":"https://api.github.com/repos/Financial-Times/next-ilyushin","forks_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/forks","keys_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/teams","hooks_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/hooks","issue_events_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/issues/events{/number}","events_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/events","assignees_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/assignees{/user}","branches_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/branches{/branch}","tags_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/tags","blobs_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/git/refs{/sha}","trees_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/statuses/{sha}","languages_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/languages","stargazers_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/stargazers","contributors_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/contributors","subscribers_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/subscribers","subscription_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/subscription","commits_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/commits{/sha}","git_commits_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/git/commits{/sha}","comments_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/comments{/number}","issue_comment_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/issues/comments{/number}","contents_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/contents/{+path}","compare_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/merges","archive_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/downloads","issues_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/issues{/number}","pulls_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls{/number}","milestones_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/milestones{/number}","notifications_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/labels{/name}","releases_url":"https://api.github.com/repos/Financial-Times/next-ilyushin/releases{/id}","created_at":"2015-01-06T13:18:01Z","updated_at":"2015-05-07T15:08:11Z","pushed_at":"2015-09-15T11:54:08Z","git_url":"git://github.com/Financial-Times/next-ilyushin.git","ssh_url":"git@github.com:Financial-Times/next-ilyushin.git","clone_url":"https://github.com/Financial-Times/next-ilyushin.git","svn_url":"https://github.com/Financial-Times/next-ilyushin","homepage":null,"size":26631,"stargazers_count":0,"watchers_count":0,"language":"JavaScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":12,"forks":0,"open_issues":12,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/301"},"html":{"href":"https://github.com/Financial-Times/next-ilyushin/pull/301"},"issue":{"href":"https://api.github.com/repos/Financial-Times/next-ilyushin/issues/301"},"comments":{"href":"https://api.github.com/repos/Financial-Times/next-ilyushin/issues/301/comments"},"review_comments":{"href":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/301/comments"},"review_comment":{"href":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Financial-Times/next-ilyushin/pulls/301/commits"},"statuses":{"href":"https://api.github.com/repos/Financial-Times/next-ilyushin/statuses/cf1de0fd94433b69b0489094a21a998735965675"}}}}