brownplt / pyret-lang

The Pyret language.
Other
1.07k stars 111 forks source link

loading spreadsheet documentation needs love #1651

Open shriram opened 2 years ago

shriram commented 2 years ago

https://www.pyret.org/docs/latest/gdrive-sheets.html does not make clear what the id field of load-spreadsheet is. In particular, it does not explain the URL-surgery that is needed to obtain an id. This is documented elsewhere but that's not linked.

Worse, the same thing does not seem to work for my-spreadsheet, which is documented as taking a name as opposed to an id. The same surgery on a URL does not seem to work, nor does the URL itself. So I'm actually unclear on how to use this.

schanzer commented 2 years ago

Load-spreadsheet should REALLY be able to take an entire google url. That function is imported from gdrive-sheets, right?

Finding the google ID is error prone even if you know what to look for.

shriram commented 2 years ago

And, if you even get so much as a character off (eg, you correctly identify the ID part but leave a slash in at the beginning or end), you'll still give an unhelpful error.