Skip to content
This repository has been archived by the owner on Dec 27, 2024. It is now read-only.

fix: set proper metadata action when an announcement is deleted #517

Merged

Conversation

mycodeself
Copy link
Contributor

@mycodeself mycodeself commented Dec 3, 2024

Summary 📰

Set proper metadata.action for the event triggered when an announcement is deleted.

Fixes #516

Details 📚

Is just a small change, instead sending the EVENTS_ACTION_DELETE_CATEGORY when an announcement is deleted now it sends the proper one EVENTS_ACTION_DELETE_ANNOUNCEMENT

Checklist ✅

  • I have updated the necessary documentation
  • I have signed off all my commits as required by DCO
  • My build is green

Testing Notes 🔬

@mycodeself mycodeself requested review from kurtaking and a team as code owners December 3, 2024 13:33
Copy link

changeset-bot bot commented Dec 3, 2024

🦋 Changeset detected

Latest commit: 4e9be51

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@procore-oss/backstage-plugin-announcements-backend Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mycodeself mycodeself force-pushed the fix/wrong-event-on-announcement-delete branch from 1a8c1e8 to c68a179 Compare December 3, 2024 13:42
github-actions bot and others added 2 commits December 3, 2024 14:43
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: mycodeself <[email protected]>
@mycodeself mycodeself force-pushed the fix/wrong-event-on-announcement-delete branch from c68a179 to d227a3f Compare December 3, 2024 13:44
Copy link
Contributor

@kurtaking kurtaking left a comment

Choose a reason for hiding this comment

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

Thanks, we should have tests that catch this 🤦🏼‍♂️

@kurtaking kurtaking merged commit 21228b4 into procore-oss:main Dec 6, 2024
3 checks passed
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.

Wrong event triggered when deleting an announcement
3 participants