Skip to content
This repository was archived by the owner on Feb 6, 2026. It is now read-only.

fix(clover): fix up sql admin users delete func with override - #8274

Merged
aaron-dernley merged 1 commit into
mainfrom
fix/sql-admin-user-delete
Jan 14, 2026
Merged

fix(clover): fix up sql admin users delete func with override#8274
aaron-dernley merged 1 commit into
mainfrom
fix/sql-admin-user-delete

Conversation

@aaron-dernley

Copy link
Copy Markdown
Contributor

This PR:

  • Fix SQL Admin Users delete failing with "Missing parameter: user" error
  • SQL Admin API requires name as a query parameter for delete (not path parameter like other APIs)
  • Added asset-specific delete override that includes name (and optional host for MySQL) as query params

How was it tested?

  • Integration tests pass
  • Manual test: new functionality works in UI

Does it require a docs change?

  • No
  • Yes, and this PR includes it
  • Yes, and this PR does not include it (reasoning below)

In short: 🔗

@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions

Copy link
Copy Markdown

Working with Module Index at: https://module-index.systeminit.com
Total: 0 new asset(s), 1 changed asset(s)

Google - 0 new, 1 changed asset(s)
[Google Cloud SQL Admin Users]: other schema contents 🔀3

@aaron-dernley
aaron-dernley marked this pull request as ready for review January 14, 2026 19:08
@aaron-dernley
aaron-dernley added this pull request to the merge queue Jan 14, 2026
Merged via the queue into main with commit 1807978 Jan 14, 2026
10 checks passed
@aaron-dernley
aaron-dernley deleted the fix/sql-admin-user-delete branch January 14, 2026 19:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants