Skip to content

Conversation

valaparthvi
Copy link
Contributor

kind/bug
What this PR does / why we need it:

This is a followup PR to fix things we missed out in #719.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

Checklist:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

@valaparthvi valaparthvi requested a review from a team as a code owner August 22, 2025 13:21
@kkaempf kkaempf moved this to PR to be reviewed in CAPI / Turtles Aug 22, 2025
Copy link
Contributor

@mjura mjura left a comment

Choose a reason for hiding this comment

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

Please squash commits into one

deletion to webhook function

Co-authored-by: Andrea Mazzotti <[email protected]>
Signed-off-by: Parthvi Vala <[email protected]>
@valaparthvi valaparthvi force-pushed the followup-additionaluserdata branch from 01b9a1b to 9206024 Compare August 26, 2025 09:31
@valaparthvi valaparthvi requested a review from mjura August 26, 2025 09:31
Copy link
Contributor

@mjura mjura left a comment

Choose a reason for hiding this comment

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

Lgtm

@alexander-demicev alexander-demicev added this pull request to the merge queue Aug 26, 2025
Merged via the queue into rancher:main with commit f7291d9 Aug 26, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from PR to be reviewed to Done in CAPI / Turtles Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants