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

tools: Update Google One-Click Deploy configuration #29

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

malexw
Copy link
Collaborator

@malexw malexw commented Apr 23, 2024

The app name previously was fairly generic, so I've updated it to be more specific to the application we're deploying.

There was also an issue with the "default" field, which doesn't actually exist in the spec, and should be "value".

@malexw malexw requested a review from a team as a code owner April 23, 2024 21:37
@CLAassistant
Copy link

CLAassistant commented Apr 23, 2024

CLA assistant check
All committers have signed the CLA.

@malexw
Copy link
Collaborator Author

malexw commented Apr 25, 2024

This needs another round of testing before we merge

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.

None yet

4 participants