Skip to content

fix(announcements): exclude inactive announcements from search results - #10334

Open
gaelgoth wants to merge 1 commit into
backstage:mainfrom
gaelgoth:fix/exlude-inactive-announcement-from-search-indexes
Open

fix(announcements): exclude inactive announcements from search results#10334
gaelgoth wants to merge 1 commit into
backstage:mainfrom
gaelgoth:fix/exlude-inactive-announcement-from-search-indexes

Conversation

@gaelgoth

Copy link
Copy Markdown
Member

Hey, I just made a Pull Request!

It fixes #10197. I've added a continue when we have an inactive announcement in the indexation loop,

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)
  • All your commits have a Signed-off-by line in the message. (more info)

Signed-off-by: gaelgoth <gothuey.gael@gmail.com>
@gaelgoth
gaelgoth requested review from a team and kurtaking as code owners August 17, 2026 09:18
@gaelgoth
gaelgoth requested a review from BethGriggs August 17, 2026 09:18
@backstage-service backstage-service added the workspace/announcements Used to tag announcements workspace isses and pull requests label Aug 17, 2026
@backstage-goalie

Copy link
Copy Markdown
Contributor

Changed Packages

Package Name Package Path Changeset Bump Current Version
@backstage-community/plugin-search-backend-module-announcements workspaces/announcements/plugins/search-backend-module-announcements minor v0.19.0

@kurtaking kurtaking left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice, this a good quality of life improvement. Thank you @gaelgoth

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

workspace/announcements Used to tag announcements workspace isses and pull requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 announcements: inactive announcements (search and detail view)

3 participants