{"action":"created","issue":{"url":"https://api.github.com/repos/opendatakit/collect/issues/688","repository_url":"https://api.github.com/repos/opendatakit/collect","labels_url":"https://api.github.com/repos/opendatakit/collect/issues/688/labels{/name}","comments_url":"https://api.github.com/repos/opendatakit/collect/issues/688/comments","events_url":"https://api.github.com/repos/opendatakit/collect/issues/688/events","html_url":"https://github.com/opendatakit/collect/pull/688","id":215201117,"number":688,"title":"Updation for issue #310","user":{"login":"shouryalala","id":18574571,"avatar_url":"https://avatars1.githubusercontent.com/u/18574571?v=3","gravatar_id":"","url":"https://api.github.com/users/shouryalala","html_url":"https://github.com/shouryalala","followers_url":"https://api.github.com/users/shouryalala/followers","following_url":"https://api.github.com/users/shouryalala/following{/other_user}","gists_url":"https://api.github.com/users/shouryalala/gists{/gist_id}","starred_url":"https://api.github.com/users/shouryalala/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shouryalala/subscriptions","organizations_url":"https://api.github.com/users/shouryalala/orgs","repos_url":"https://api.github.com/users/shouryalala/repos","events_url":"https://api.github.com/users/shouryalala/events{/privacy}","received_events_url":"https://api.github.com/users/shouryalala/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-03-18T15:27:06Z","updated_at":"2017-03-21T07:05:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/opendatakit/collect/pulls/688","html_url":"https://github.com/opendatakit/collect/pull/688","diff_url":"https://github.com/opendatakit/collect/pull/688.diff","patch_url":"https://github.com/opendatakit/collect/pull/688.patch"},"body":"Hi @yanokwa, apologies for the late update. Exams have had the best of me.\r\n\r\nOkay so after going through a few cases this is what i gathered\r\n1. The submission url in the form always takes precedence over the GoogleSheetsURL field\r\n2. The 'uploadOneSubmission' method in the GoogleSheetsAbstractUploader class specifies that only urls' of the form \"../docs.google.com/spreadsheets/d/...\" will be accepted. It makes the verification of the url more easier but i feel that there is a trade off. If a person is present at site and needs to send a form to a separate spreadsheet, he/she might need to use a shortened url(goo.gl/, bit.do/, etc ) as it really isnt possible to remember a spreadsheet url. But this can be taken up later i guess. \r\n3. The code otherwise is very robust. Any Google spreadsheet can be used to upload form data as long as the first row of the sheet is empty.\r\n4. If there is no url provided in either the form or the settings field, it gives an invalid sheet id message.\r\n\r\nThe quick addition i made is that i added an appropriate hint text in the settings GoogleSheetsUrl field:- \r\n\"http://docs.google.com/spreadsheets/d/sheet_id\"\r\n\r\nNote: There are no changes in the readme. I had edited it in the master branch by accident while looking into a different issue "},"comment":{"url":"https://api.github.com/repos/opendatakit/collect/issues/comments/287993760","html_url":"https://github.com/opendatakit/collect/pull/688#issuecomment-287993760","issue_url":"https://api.github.com/repos/opendatakit/collect/issues/688","id":287993760,"user":{"login":"shobhitagarwal1612","id":8918023,"avatar_url":"https://avatars2.githubusercontent.com/u/8918023?v=3","gravatar_id":"","url":"https://api.github.com/users/shobhitagarwal1612","html_url":"https://github.com/shobhitagarwal1612","followers_url":"https://api.github.com/users/shobhitagarwal1612/followers","following_url":"https://api.github.com/users/shobhitagarwal1612/following{/other_user}","gists_url":"https://api.github.com/users/shobhitagarwal1612/gists{/gist_id}","starred_url":"https://api.github.com/users/shobhitagarwal1612/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shobhitagarwal1612/subscriptions","organizations_url":"https://api.github.com/users/shobhitagarwal1612/orgs","repos_url":"https://api.github.com/users/shobhitagarwal1612/repos","events_url":"https://api.github.com/users/shobhitagarwal1612/events{/privacy}","received_events_url":"https://api.github.com/users/shobhitagarwal1612/received_events","type":"User","site_admin":false},"created_at":"2017-03-21T07:05:37Z","updated_at":"2017-03-21T07:05:37Z","body":"@shouryalala I think a regex for the required sheets url should be written and match the inputted data with that regex. If it doesn't match, then display error message that the url is not well defined"}}
{"actor":{"display_login":"shobhitagarwal1612"}}