{"action":"created","issue":{"url":"https://api.github.com/repos/zanematthew/zm-ajax-login-register/issues/108","labels_url":"https://api.github.com/repos/zanematthew/zm-ajax-login-register/issues/108/labels{/name}","comments_url":"https://api.github.com/repos/zanematthew/zm-ajax-login-register/issues/108/comments","events_url":"https://api.github.com/repos/zanematthew/zm-ajax-login-register/issues/108/events","html_url":"https://github.com/zanematthew/zm-ajax-login-register/pull/108","id":85696324,"number":108,"title":"Facebook avatars support","user":{"login":"timersys","id":1206915,"avatar_url":"https://avatars.githubusercontent.com/u/1206915?v=3","gravatar_id":"","url":"https://api.github.com/users/timersys","html_url":"https://github.com/timersys","followers_url":"https://api.github.com/users/timersys/followers","following_url":"https://api.github.com/users/timersys/following{/other_user}","gists_url":"https://api.github.com/users/timersys/gists{/gist_id}","starred_url":"https://api.github.com/users/timersys/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/timersys/subscriptions","organizations_url":"https://api.github.com/users/timersys/orgs","repos_url":"https://api.github.com/users/timersys/repos","events_url":"https://api.github.com/users/timersys/events{/privacy}","received_events_url":"https://api.github.com/users/timersys/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-06-06T04:45:13Z","updated_at":"2015-06-08T15:17:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/zanematthew/zm-ajax-login-register/pulls/108","html_url":"https://github.com/zanematthew/zm-ajax-login-register/pull/108","diff_url":"https://github.com/zanematthew/zm-ajax-login-register/pull/108.diff","patch_url":"https://github.com/zanematthew/zm-ajax-login-register/pull/108.patch"},"body":"Hi Zane again, I added facebook avatar support, but I have a few comments here\r\n\r\n* Meta tags file doesn't make much sense to me the way it was. App id was not loading for me so I changed it a bit but probably Im missing something, its 1am here :)\r\n* on login.php I adder a user_meta that will identify normal users from facebook users. \r\n* On abstract I added the new setting but can be removed if you want as get_avatar will fallback to normal avatar if is not a fb user. I just though would be nice to give user an option\r\n\r\nI will probably send more changes later this week, Im using the plugin for a project."},"comment":{"url":"https://api.github.com/repos/zanematthew/zm-ajax-login-register/issues/comments/110035753","html_url":"https://github.com/zanematthew/zm-ajax-login-register/pull/108#issuecomment-110035753","issue_url":"https://api.github.com/repos/zanematthew/zm-ajax-login-register/issues/108","id":110035753,"user":{"login":"timersys","id":1206915,"avatar_url":"https://avatars.githubusercontent.com/u/1206915?v=3","gravatar_id":"","url":"https://api.github.com/users/timersys","html_url":"https://github.com/timersys","followers_url":"https://api.github.com/users/timersys/followers","following_url":"https://api.github.com/users/timersys/following{/other_user}","gists_url":"https://api.github.com/users/timersys/gists{/gist_id}","starred_url":"https://api.github.com/users/timersys/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/timersys/subscriptions","organizations_url":"https://api.github.com/users/timersys/orgs","repos_url":"https://api.github.com/users/timersys/repos","events_url":"https://api.github.com/users/timersys/events{/privacy}","received_events_url":"https://api.github.com/users/timersys/received_events","type":"User","site_admin":false},"created_at":"2015-06-08T15:17:37Z","updated_at":"2015-06-08T15:17:37Z","body":"Hi yes cherry picking commits it's a bit complicate. In order to create different pull request I had to start branchs on your last commit which didn't have the others changes yet.\r\n\r\nI will investigate this afternoon how to modify it now that the other commits are already merged.\r\n"}}