{"action":"created","issue":{"url":"https://api.github.com/repos/astropy/astroquery/issues/1288","repository_url":"https://api.github.com/repos/astropy/astroquery","labels_url":"https://api.github.com/repos/astropy/astroquery/issues/1288/labels{/name}","comments_url":"https://api.github.com/repos/astropy/astroquery/issues/1288/comments","events_url":"https://api.github.com/repos/astropy/astroquery/issues/1288/events","html_url":"https://github.com/astropy/astroquery/pull/1288","id":383117706,"node_id":"MDExOlB1bGxSZXF1ZXN0MjMyNjYzMTA5","number":1288,"title":"[WIP] Adding esa_hubble module","user":{"login":"esdc-esac-esa-int","id":13047336,"node_id":"MDQ6VXNlcjEzMDQ3MzM2","avatar_url":"https://avatars0.githubusercontent.com/u/13047336?v=4","gravatar_id":"","url":"https://api.github.com/users/esdc-esac-esa-int","html_url":"https://github.com/esdc-esac-esa-int","followers_url":"https://api.github.com/users/esdc-esac-esa-int/followers","following_url":"https://api.github.com/users/esdc-esac-esa-int/following{/other_user}","gists_url":"https://api.github.com/users/esdc-esac-esa-int/gists{/gist_id}","starred_url":"https://api.github.com/users/esdc-esac-esa-int/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/esdc-esac-esa-int/subscriptions","organizations_url":"https://api.github.com/users/esdc-esac-esa-int/orgs","repos_url":"https://api.github.com/users/esdc-esac-esa-int/repos","events_url":"https://api.github.com/users/esdc-esac-esa-int/events{/privacy}","received_events_url":"https://api.github.com/users/esdc-esac-esa-int/received_events","type":"User","site_admin":false},"labels":[{"id":35934983,"node_id":"MDU6TGFiZWwzNTkzNDk4Mw==","url":"https://api.github.com/repos/astropy/astroquery/labels/New%20Service","name":"New Service","color":"e102d8","default":false},{"id":692578148,"node_id":"MDU6TGFiZWw2OTI1NzgxNDg=","url":"https://api.github.com/repos/astropy/astroquery/labels/Work%20in%20progress","name":"Work in progress","color":"f9d0c4","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-11-21T13:27:18Z","updated_at":"2018-11-22T16:14:15Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/astropy/astroquery/pulls/1288","html_url":"https://github.com/astropy/astroquery/pull/1288","diff_url":"https://github.com/astropy/astroquery/pull/1288.diff","patch_url":"https://github.com/astropy/astroquery/pull/1288.patch"},"body":"Adding first version of the 'esa_hubble' module"},"comment":{"url":"https://api.github.com/repos/astropy/astroquery/issues/comments/441075021","html_url":"https://github.com/astropy/astroquery/pull/1288#issuecomment-441075021","issue_url":"https://api.github.com/repos/astropy/astroquery/issues/1288","id":441075021,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0MTA3NTAyMQ==","user":{"login":"jcsegovia","id":19604543,"node_id":"MDQ6VXNlcjE5NjA0NTQz","avatar_url":"https://avatars2.githubusercontent.com/u/19604543?v=4","gravatar_id":"","url":"https://api.github.com/users/jcsegovia","html_url":"https://github.com/jcsegovia","followers_url":"https://api.github.com/users/jcsegovia/followers","following_url":"https://api.github.com/users/jcsegovia/following{/other_user}","gists_url":"https://api.github.com/users/jcsegovia/gists{/gist_id}","starred_url":"https://api.github.com/users/jcsegovia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jcsegovia/subscriptions","organizations_url":"https://api.github.com/users/jcsegovia/orgs","repos_url":"https://api.github.com/users/jcsegovia/repos","events_url":"https://api.github.com/users/jcsegovia/events{/privacy}","received_events_url":"https://api.github.com/users/jcsegovia/received_events","type":"User","site_admin":false},"created_at":"2018-11-22T16:14:15Z","updated_at":"2018-11-22T16:14:15Z","author_association":"CONTRIBUTOR","body":"@jduran I have been reviewing your code.\r\n- When executing methods that imply data download, I think it is useful to inform the user that a file has been created.\r\n- Please, move Conf initialization to __init__.py\r\n- Please, change private attributes names: replace double underscores by single ones.\r\n- Please,  check point 4 in doc because it seems it is not working. "}}
{"actor":{"display_login":"jcsegovia"}}