Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(notifications): add title to the Notifications' table #27568

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

mareklibra
Copy link
Contributor

Hey, I just made a Pull Request!

Notifications' table has newly a title with count of filtered records.

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

@mareklibra mareklibra requested review from a team as code owners November 9, 2024 21:36
@backstage-goalie
Copy link
Contributor

backstage-goalie bot commented Nov 9, 2024

Changed Packages

Package Name Package Path Changeset Bump Current Version
@backstage/plugin-notifications plugins/notifications patch v0.3.3-next.2

@mareklibra
Copy link
Contributor Author

unread


saved

Copy link
Member

@jhaals jhaals left a comment

Choose a reason for hiding this comment

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

Looking good! Minor tweaks and this should be good to go 👏

'@backstage/plugin-notifications': minor
---

Added title to the Notifications' table. The title clearly states the filter selectino and total count of messages. This change aligns the look and fel closer to other tables.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Added title to the Notifications' table. The title clearly states the filter selectino and total count of messages. This change aligns the look and fel closer to other tables.
Added title to the Notifications' table. The title clearly states the filter selectin and total count of messages. This change aligns the look and feel closer to other tables.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -0,0 +1,5 @@
---
'@backstage/plugin-notifications': minor
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
'@backstage/plugin-notifications': minor
'@backstage/plugin-notifications': patch

Package is below 1.x so a patch changeset is enough for introducing new non breaking features.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Signed-off-by: Marek Libra <marek.libra@gmail.com>
Copy link
Member

@jhaals jhaals left a comment

Choose a reason for hiding this comment

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

Great!

@jhaals jhaals merged commit 0adaafc into backstage:master Nov 13, 2024
25 checks passed
Copy link
Contributor

Thank you for contributing to Backstage! The changes in this pull request will be part of the 1.33.0 release, scheduled for Tue, 19 Nov 2024.

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.

2 participants