Skip to content

feat: Publish key change event after key deletion#5200

Merged
judehung merged 1 commit intoedgexfoundry:mainfrom
FelixTing:issue-5194
Jul 2, 2025
Merged

feat: Publish key change event after key deletion#5200
judehung merged 1 commit intoedgexfoundry:mainfrom
FelixTing:issue-5194

Conversation

@FelixTing
Copy link
Member

fix: #5194

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md

PR Checklist

Please check if your PR fulfills the following requirements:

  • I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
  • I am not introducing a new dependency (add notes below if you are)
  • I have added unit tests for the new feature or bug fix (if not, why?)
  • I have fully tested (add details below) this the new feature or bug fix (if not, why?)
  • I have opened a PR for the related docs change (if not, why?)

@FelixTing FelixTing requested a review from judehung July 2, 2025 01:28
Signed-off-by: FelixTing <felix@iotechsys.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 2, 2025

Copy link
Member

@judehung judehung left a comment

Choose a reason for hiding this comment

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

LGTM

@judehung judehung merged commit 37d2bc2 into edgexfoundry:main Jul 2, 2025
7 checks passed
@FelixTing FelixTing deleted the issue-5194 branch July 3, 2025 05:35
FelixTing added a commit to FelixTing/edgex-go that referenced this pull request Oct 15, 2025
Signed-off-by: FelixTing <felix@iotechsys.com>
FelixTing added a commit to FelixTing/edgex-go that referenced this pull request Oct 15, 2025
Signed-off-by: FelixTing <felix@iotechsys.com>
FelixTing added a commit to FelixTing/edgex-go that referenced this pull request Oct 15, 2025
Signed-off-by: FelixTing <felix@iotechsys.com>
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.

Publish key change event after deleting keys via CoreKeeper API

2 participants