Skip to content
This repository has been archived by the owner on Feb 12, 2023. It is now read-only.

Support underscores on column names in Google Spreadsheets #443

Open
ggalmazor opened this issue Mar 17, 2019 · 5 comments
Open

Support underscores on column names in Google Spreadsheets #443

ggalmazor opened this issue Mar 17, 2019 · 5 comments

Comments

@ggalmazor
Copy link
Contributor

As per conversation on the forums, it looks like we're replacing allegedly unsafe characters, including: *, :, -, and _ in column names when publishing to Google Spreadsheets.

We should try to understand why this is being currently done and whether we need to continue doing it or not.

@yanokwa
Copy link
Member

yanokwa commented Mar 17, 2019

@ggalmazor Where in the code is this done? Maybe git blame would help figure this out. If not, I'm hoping this is an easy thing for you to test. Not a big priority though...

@wbrunette Do you know off the top of your head?

@ggalmazor
Copy link
Contributor Author

ggalmazor commented Mar 26, 2019

I see a Dylan Price on the git blame. Does that ring a bell?

This came in 4c8abfc: "Fixed GoogleSpreadsheet trouble with special characters in headers and added parent-uid to repeat spreadsheets."
Diff: 4c8abfc?diff=unified&w=1

@yanokwa
Copy link
Member

yanokwa commented Mar 26, 2019

Yeah, I know Dylan, but I think it's too much work to track him down. We should just try underscores and special characters and update the code accordingly.

@ggalmazor ggalmazor added this to the v2.1.0 milestone Mar 27, 2019
@ggalmazor
Copy link
Contributor Author

OK, I'm tagging this issue for v2.1.0 so that we don't forget to give it a go before dismissing or fixing it.

@wbrunette
Copy link
Contributor

wbrunette commented Mar 29, 2019

Apologies for the delayed response. Old history from memory, not sure what is in the code now and haven't looked.

Column headings were originally the variable names in the xform (similar to how database columns operate). A feature request that got added was to change to the question vs the variable names in the spreadsheet.

Back to dashes and underscores ... 8-9 years ago Google spreadsheets did not support either dashes or underscores (can't remember which way it was) being in the column name and to make things complicated Google fusion tables did not accept the other, so we converted the dashes to underscores and underscores to dashes depending on which service we were exporting to based on the naming requirements of the Google service.

Statements in this comment are from memory recollection and have not been verified.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants