{"action":"created","issue":{"url":"https://api.github.com/repos/randyzwitch/RSiteCatalyst/issues/182","repository_url":"https://api.github.com/repos/randyzwitch/RSiteCatalyst","labels_url":"https://api.github.com/repos/randyzwitch/RSiteCatalyst/issues/182/labels{/name}","comments_url":"https://api.github.com/repos/randyzwitch/RSiteCatalyst/issues/182/comments","events_url":"https://api.github.com/repos/randyzwitch/RSiteCatalyst/issues/182/events","html_url":"https://github.com/randyzwitch/RSiteCatalyst/issues/182","id":177881628,"number":182,"title":"Evaluate merging GetReport and GetReportByIds together into single function","user":{"login":"randyzwitch","id":2762787,"avatar_url":"https://avatars.githubusercontent.com/u/2762787?v=3","gravatar_id":"","url":"https://api.github.com/users/randyzwitch","html_url":"https://github.com/randyzwitch","followers_url":"https://api.github.com/users/randyzwitch/followers","following_url":"https://api.github.com/users/randyzwitch/following{/other_user}","gists_url":"https://api.github.com/users/randyzwitch/gists{/gist_id}","starred_url":"https://api.github.com/users/randyzwitch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/randyzwitch/subscriptions","organizations_url":"https://api.github.com/users/randyzwitch/orgs","repos_url":"https://api.github.com/users/randyzwitch/repos","events_url":"https://api.github.com/users/randyzwitch/events{/privacy}","received_events_url":"https://api.github.com/users/randyzwitch/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/randyzwitch/RSiteCatalyst/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/randyzwitch/RSiteCatalyst/labels/question","name":"question","color":"cc317c"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-09-19T19:59:27Z","updated_at":"2016-09-21T15:15:08Z","closed_at":null,"body":"@shashispace - \r\n\r\nIn looking at the code prior to releasing a new CRAN version, it seems to me that these functions are essentially the same, one taking a single id and one taking multiple ids. Is this true, or is there something else I'm missing?\r\n\r\nIf they are basically the same function, we should deprecate one of them so that there is only one way to do things."},"comment":{"url":"https://api.github.com/repos/randyzwitch/RSiteCatalyst/issues/comments/248643786","html_url":"https://github.com/randyzwitch/RSiteCatalyst/issues/182#issuecomment-248643786","issue_url":"https://api.github.com/repos/randyzwitch/RSiteCatalyst/issues/182","id":248643786,"user":{"login":"mrshashi","id":971558,"avatar_url":"https://avatars.githubusercontent.com/u/971558?v=3","gravatar_id":"","url":"https://api.github.com/users/mrshashi","html_url":"https://github.com/mrshashi","followers_url":"https://api.github.com/users/mrshashi/followers","following_url":"https://api.github.com/users/mrshashi/following{/other_user}","gists_url":"https://api.github.com/users/mrshashi/gists{/gist_id}","starred_url":"https://api.github.com/users/mrshashi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrshashi/subscriptions","organizations_url":"https://api.github.com/users/mrshashi/orgs","repos_url":"https://api.github.com/users/mrshashi/repos","events_url":"https://api.github.com/users/mrshashi/events{/privacy}","received_events_url":"https://api.github.com/users/mrshashi/received_events","type":"User","site_admin":false},"created_at":"2016-09-21T15:15:08Z","updated_at":"2016-09-21T15:15:08Z","body":"Though my recommendation would be to leave both functions in there - it's not breaking DRY principal, and both have a slightly different use case and signatures and return values. It would be annoying for folks to try to extract report from list of lists, when all they want is a single report. (Having said that, single report could just be fetched from Queue* functions, without using the async feature.)"}}
{"actor":{"display_login":"mrshashi"}}