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

Update README instructions for copying spreadsheet #315

Open
acharliekelly opened this issue Jun 12, 2019 · 3 comments
Open

Update README instructions for copying spreadsheet #315

acharliekelly opened this issue Jun 12, 2019 · 3 comments
Assignees

Comments

@acharliekelly
Copy link
Collaborator

acharliekelly commented Jun 12, 2019

The instructions for creating your own spreadsheet don't work - copying the spreadsheet works, but since we started using the Directory, the API no longer takes the code for a single sheet. So, if you have to do database testing, I think you need write access to one of the test pages on the directory. This should be explained on the README.

@acharliekelly
Copy link
Collaborator Author

Oh, actually, never mind

@acharliekelly
Copy link
Collaborator Author

OK, now I'm not sure whether this is an issue or not.

Using the environment variable REACT_APP_GOOGLE_SHEETS_ID may or may not work - I can't get it to run locally, so I can't test it. But there is code in AppContainer.js that should use the sheet ID, if one is provided.
Does anyone know if this actually works?

@acharliekelly acharliekelly reopened this Jun 12, 2019
@acharliekelly acharliekelly added the discussion needed ⚠️ This issue needs to be discussed further before we can begin work on it label Jun 12, 2019
@acharliekelly acharliekelly removed the discussion needed ⚠️ This issue needs to be discussed further before we can begin work on it label Jun 19, 2019
@acharliekelly
Copy link
Collaborator Author

We don't need the environment variable anymore. Just put the new spreadsheet on the Directory, and then point your app to that resource's path. But it's possible someone might still want to use the env option.

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

No branches or pull requests

2 participants