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

fix issue #19621 #21325

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

Conversation

stefan-malin-db
Copy link

No description provided.

@stefan-malin-db stefan-malin-db requested a review from a team as a code owner February 10, 2025 11:41
Copy link

** WARNING **: This repository is generated by https://github.com/GoogleCloudPlatform/magic-modules. Any changes made directly to this repository will likely be overwritten. If you have further questions, please feel free to ping your reviewer or, internal employees, reach out to one of the engineers. Thank you!

@github-actions github-actions bot requested a review from SirGitsalot February 10, 2025 11:42
@stefan-malin-ing
Copy link

@SirGitsalot can you please advise on changes?
@fd @ndarilek @mitchellh

@ndarilek
Copy link
Contributor

@stefan-malin-ing Why did you mention me? I have nothing to do with this issue at all. Why spam me with it?

@SirGitsalot
Copy link
Collaborator

Sorry for the delay - this is the wrong repo for this change as these files are generated from the Magic Modules repo. For instance, for the change to google_discovery_engine_data_store you'll need to edit this config. There's detailed instructions available. Sorry for the confusion!

@smalin94
Copy link

@stefan-malin-ing Why did you mention me? I have nothing to do with this issue at all. Why spam me with it?

Sorry, not intended!

@smalin94
Copy link

Sorry for the delay - this is the wrong repo for this change as these files are generated from the Magic Modules repo. For instance, for the change to google_discovery_engine_data_store you'll need to edit this config. There's detailed instructions available. Sorry for the confusion!

Thank you, will close it this case!

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.

5 participants