{"action":"created","issue":{"url":"https://api.github.com/repos/Automattic/wp-calypso/issues/22509","repository_url":"https://api.github.com/repos/Automattic/wp-calypso","labels_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/22509/labels{/name}","comments_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/22509/comments","events_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/22509/events","html_url":"https://github.com/Automattic/wp-calypso/pull/22509","id":297620824,"number":22509,"title":"Jetpack Connect: Fetch only the connected site after authorize","user":{"login":"seear","id":7767559,"avatar_url":"https://avatars3.githubusercontent.com/u/7767559?v=4","gravatar_id":"","url":"https://api.github.com/users/seear","html_url":"https://github.com/seear","followers_url":"https://api.github.com/users/seear/followers","following_url":"https://api.github.com/users/seear/following{/other_user}","gists_url":"https://api.github.com/users/seear/gists{/gist_id}","starred_url":"https://api.github.com/users/seear/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seear/subscriptions","organizations_url":"https://api.github.com/users/seear/orgs","repos_url":"https://api.github.com/users/seear/repos","events_url":"https://api.github.com/users/seear/events{/privacy}","received_events_url":"https://api.github.com/users/seear/received_events","type":"User","site_admin":false},"labels":[{"id":285009982,"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/API","name":"API","color":"bfdadc","default":false},{"id":285009991,"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/Jetpack","name":"Jetpack","color":"006b75","default":false},{"id":520550213,"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/Performance","name":"Performance","color":"bfdadc","default":false},{"id":285010018,"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/[Status]%20Needs%20Review","name":"[Status] Needs Review","color":"fbca04","default":false},{"id":838866001,"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/[Status]%20Needs%20e2e%20Testing%20(BETA)","name":"[Status] Needs e2e Testing (BETA)","color":"ed74b2","default":false},{"id":285009976,"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/[Type]%20Enhancement","name":"[Type] Enhancement","color":"c7def8","default":false}],"state":"open","locked":false,"assignee":{"login":"seear","id":7767559,"avatar_url":"https://avatars3.githubusercontent.com/u/7767559?v=4","gravatar_id":"","url":"https://api.github.com/users/seear","html_url":"https://github.com/seear","followers_url":"https://api.github.com/users/seear/followers","following_url":"https://api.github.com/users/seear/following{/other_user}","gists_url":"https://api.github.com/users/seear/gists{/gist_id}","starred_url":"https://api.github.com/users/seear/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seear/subscriptions","organizations_url":"https://api.github.com/users/seear/orgs","repos_url":"https://api.github.com/users/seear/repos","events_url":"https://api.github.com/users/seear/events{/privacy}","received_events_url":"https://api.github.com/users/seear/received_events","type":"User","site_admin":false},"assignees":[{"login":"seear","id":7767559,"avatar_url":"https://avatars3.githubusercontent.com/u/7767559?v=4","gravatar_id":"","url":"https://api.github.com/users/seear","html_url":"https://github.com/seear","followers_url":"https://api.github.com/users/seear/followers","following_url":"https://api.github.com/users/seear/following{/other_user}","gists_url":"https://api.github.com/users/seear/gists{/gist_id}","starred_url":"https://api.github.com/users/seear/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seear/subscriptions","organizations_url":"https://api.github.com/users/seear/orgs","repos_url":"https://api.github.com/users/seear/repos","events_url":"https://api.github.com/users/seear/events{/privacy}","received_events_url":"https://api.github.com/users/seear/received_events","type":"User","site_admin":false}],"milestone":null,"comments":5,"created_at":"2018-02-15T22:32:56Z","updated_at":"2018-02-21T14:01:42Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/Automattic/wp-calypso/pulls/22509","html_url":"https://github.com/Automattic/wp-calypso/pull/22509","diff_url":"https://github.com/Automattic/wp-calypso/pull/22509.diff","patch_url":"https://github.com/Automattic/wp-calypso/pull/22509.patch"},"body":"Instead of all the sites! We need to force the API to fetch from wpcom instead of the .org site, which may not yet be accessible.\r\n\r\n## Testing\r\n* Check that a site can still be connected at /jetpack/connect, or from wp-admin\r\n* The newly connected site should be present in the sites list in the calypso sidebar after connection\r\n* Check the other connection flows listed at PCYsg-dek-p2\r\n"},"comment":{"url":"https://api.github.com/repos/Automattic/wp-calypso/issues/comments/367334894","html_url":"https://github.com/Automattic/wp-calypso/pull/22509#issuecomment-367334894","issue_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/22509","id":367334894,"user":{"login":"seear","id":7767559,"avatar_url":"https://avatars3.githubusercontent.com/u/7767559?v=4","gravatar_id":"","url":"https://api.github.com/users/seear","html_url":"https://github.com/seear","followers_url":"https://api.github.com/users/seear/followers","following_url":"https://api.github.com/users/seear/following{/other_user}","gists_url":"https://api.github.com/users/seear/gists{/gist_id}","starred_url":"https://api.github.com/users/seear/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seear/subscriptions","organizations_url":"https://api.github.com/users/seear/orgs","repos_url":"https://api.github.com/users/seear/repos","events_url":"https://api.github.com/users/seear/events{/privacy}","received_events_url":"https://api.github.com/users/seear/received_events","type":"User","site_admin":false},"created_at":"2018-02-21T14:01:42Z","updated_at":"2018-02-21T14:01:42Z","author_association":"MEMBER","body":"> An option we discussed elsewhere would be to roll this out \"progressively\" as an AB test where we can monitor the performance before rolling out for everyone. Do you think there's reason to be so cautious?\r\n\r\nI love the idea of progressive roll out, and I've made changes before where I had wished we could do something like that.\r\n\r\nHowever, I'm not sure that this change warrants that extra complexity, because:\r\n1. As it stands I don't think this PR is too risky. We are replacing a fetch of all sites with a known-good way to fetch a single site.\r\n1. Would a progressive roll out actually make it harder to spot any dips in the numbers?\r\n\r\nSo I suggest we merge when ready and keep an eye on the funnels.\r\n"}}
{"actor":{"display_login":"seear"}}