{"action":"created","issue":{"url":"https://api.github.com/repos/plotly/documentation/issues/176","labels_url":"https://api.github.com/repos/plotly/documentation/issues/176/labels{/name}","comments_url":"https://api.github.com/repos/plotly/documentation/issues/176/comments","events_url":"https://api.github.com/repos/plotly/documentation/issues/176/events","html_url":"https://github.com/plotly/documentation/pull/176","id":101304749,"number":176,"title":"Api doc parity advanced charts","user":{"login":"NicoleGrondin","id":10135076,"avatar_url":"https://avatars.githubusercontent.com/u/10135076?v=3","gravatar_id":"","url":"https://api.github.com/users/NicoleGrondin","html_url":"https://github.com/NicoleGrondin","followers_url":"https://api.github.com/users/NicoleGrondin/followers","following_url":"https://api.github.com/users/NicoleGrondin/following{/other_user}","gists_url":"https://api.github.com/users/NicoleGrondin/gists{/gist_id}","starred_url":"https://api.github.com/users/NicoleGrondin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NicoleGrondin/subscriptions","organizations_url":"https://api.github.com/users/NicoleGrondin/orgs","repos_url":"https://api.github.com/users/NicoleGrondin/repos","events_url":"https://api.github.com/users/NicoleGrondin/events{/privacy}","received_events_url":"https://api.github.com/users/NicoleGrondin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-08-16T22:22:16Z","updated_at":"2015-08-17T20:01:35Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/plotly/documentation/pulls/176","html_url":"https://github.com/plotly/documentation/pull/176","diff_url":"https://github.com/plotly/documentation/pull/176.diff","patch_url":"https://github.com/plotly/documentation/pull/176.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/plotly/documentation/issues/comments/131944536","html_url":"https://github.com/plotly/documentation/pull/176#issuecomment-131944536","issue_url":"https://api.github.com/repos/plotly/documentation/issues/176","id":131944536,"user":{"login":"jackparmer","id":1865834,"avatar_url":"https://avatars.githubusercontent.com/u/1865834?v=3","gravatar_id":"","url":"https://api.github.com/users/jackparmer","html_url":"https://github.com/jackparmer","followers_url":"https://api.github.com/users/jackparmer/followers","following_url":"https://api.github.com/users/jackparmer/following{/other_user}","gists_url":"https://api.github.com/users/jackparmer/gists{/gist_id}","starred_url":"https://api.github.com/users/jackparmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jackparmer/subscriptions","organizations_url":"https://api.github.com/users/jackparmer/orgs","repos_url":"https://api.github.com/users/jackparmer/repos","events_url":"https://api.github.com/users/jackparmer/events{/privacy}","received_events_url":"https://api.github.com/users/jackparmer/received_events","type":"User","site_admin":false},"created_at":"2015-08-17T20:01:35Z","updated_at":"2015-08-17T20:01:35Z","body":"i made a quick d3.csv example:\nhttp://codepen.io/plotly/pen/9a091a6dd0486ff8de95982bc23a3467?editors=101\n\nOn Mon, Aug 17, 2015 at 3:57 PM, chriddyp <notifications@github.com> wrote:\n\n> otherwise looks good! in the future, a few things changes to how you\n> commit would really help review for big PRs like this. Viewing \"Files\n> changed\" almost crashes my browser!\n>\n>    - Keep commits to one thing: most of the changes to the 175 files were\n>    just white space removals. it would be great if all of those changes were\n>    in a single commit: 'removed white space' so that I can just skip reviewing\n>    that. similarly with changes to adding 'horizontal' - keeping this as\n>    a single commit with only those changes makes review a lot faster\n>    - A couple commits had changes to _config_dev.yml that looked like\n>    they were just used in development - you don't need to commit all of the\n>    changes that you make everything\n>    - keep commit messages super descriptive. the \"more advanced charts\"\n>    commit doesn't look like it actually added any more examples, it just\n>    cleaned up some of the white spaces and urls\n>\n> I use the github mac client: https://desktop.github.com, and it makes\n> organizing these commits and pushes really really easy and nice! lemme know\n> if you have any questions :)\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/plotly/documentation/pull/176#issuecomment-131943216>.\n>\n"}}