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

.clasp.json will be overwrited for people using clasp to upload GAS to their own projects #431

Open
brentchang opened this issue Dec 3, 2023 · 2 comments
Assignees

Comments

@brentchang
Copy link

brentchang commented Dec 3, 2023

Summary

The .clasp.json files contain the GAS project IDs only accessible inside Google. (example)
However, people using clasp CLI to push to their GAS project ID will need to overwrite these files.

What I think it might be better in either way:

  1. add .clasp.json in .gitignore file but it will not work since this file is required for your Github Actions Workflow.
  2. rename the Google-only accessible .clasp.json to other filename instead of default .clasp.json, while rename them back to .clasp.json before executing clasp push -f(link) inside your Github Action Workflow

Expected Behavior

People not work at Google can clone/commit this repo without overwriting this .clasp.json file.

Actual Behavior

Everyone outside Google needs to overwrite this .clasp.json file to run the clasp push to their GAS project.

Steps to Reproduce the Problem

  1. login with personal gmail account
  2. create a new Google Sheet and retrieve the GAS project ID through Extensions > Apps Script menu
  3. Go to Project Settings to retrieve Script ID
  4. Overwrite the scriptID within solutions/automations/mail-merge/.clasp.google-only.json
  5. run clasp push under solutions/automations/mail-merge/ directory

But I have no idea how Googlers use these .clasp.json files, so feel free to provide any other idea, thanks!

@o23bsr0620-capke
Copy link

Has solutions??

@brentchang
Copy link
Author

brentchang commented Dec 12, 2023

Yes, based on my second idea, I created the PR as a solution.

#432

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

No branches or pull requests

3 participants