Skip to content

Conversation

@Sew9
Copy link

@Sew9 Sew9 commented Nov 13, 2025

Hey, I just made a Pull Request!

Changed the AnnouncementsPage to hide the 'New announcement' button when users lack create permission, rather than showing it in a disabled state. This provides a cleaner UI experience for users without the necessary permissions.

Added tests to verify the button visibility based on permissions.

after:
image
image

before:
image
image

✔️ 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)

Changed the AnnouncementsPage to hide the 'New announcement' button when users lack create permission, rather than showing it in a disabled state. This provides a cleaner UI experience for users without the necessary permissions.

Added tests to verify the button visibility based on permissions.

Signed-off-by: sew9 <[email protected]>
@Sew9 Sew9 requested review from a team, gaelgoth and kurtaking as code owners November 13, 2025 07:06
@Sew9 Sew9 requested a review from BethGriggs November 13, 2025 07:06
@backstage-service backstage-service added the workspace/announcements Used to tag announcements workspace isses and pull requests label Nov 13, 2025
@backstage-goalie
Copy link
Contributor

Changed Packages

Package Name Package Path Changeset Bump Current Version
@backstage-community/plugin-announcements workspaces/announcements/plugins/announcements patch v0.16.0

Copy link
Member

@gaelgoth gaelgoth left a comment

Choose a reason for hiding this comment

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

Hi @Sew9, thanks for this!

Thinking out loud here; when setting up announcement permissions, having the button disabled could help confirm that the permissions are configured correctly. Maybe we could introduce a prop/config option that hides the button based on permissions if needed?

Would love to hear your thoughts on this as well, @kurtaking.

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.

3 participants