Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Google Sheets integration for progression updates #967

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

petterhj
Copy link
Collaborator

@petterhj petterhj commented Apr 5, 2024

No description provided.

@petterhj petterhj requested a review from a team April 5, 2024 12:35
CHANGELOG.md Outdated Show resolved Hide resolved
Comment on lines 71 to 78
- From the **Project Overview**, select **Service accounts**
- Click **Generate new private key**

This key is used for fetching data from Google Sheets (for automatically updating key results). In order to fetch data from Google Sheets, you must set up environment variables for Firebase Functions:

```bash
firebase functions:config:set
service_account="<service account private key json-file>"
storage.bucket="<your-storage-bucket-name>"
sheets.impersonator="email-address" (optional)
```
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mulig at hele denne service accounten kan fjernes nå?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ja, tror muligens det, og tror kanskje ikke den ev. må konfigureres i initializeApp heller. Det er default-accounten som lages når prosjektet opprettes. Men tenkte egentlig å undersøke litt nærmere og ev. rydde bort mer i egen runde.

@petterhj petterhj force-pushed the remove-google-sheets branch from aba119a to 1b693a9 Compare April 9, 2024 07:05
@petterhj petterhj merged commit 00bde97 into main Apr 11, 2024
3 checks passed
@petterhj petterhj deleted the remove-google-sheets branch April 11, 2024 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants