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

Added emptyContent to properties of Catalog Table #14003

Merged
merged 5 commits into from Oct 17, 2022

Conversation

Irma12
Copy link
Contributor

@Irma12 Irma12 commented Oct 5, 2022

Signed-off-by: irma12 irma@roadie.io

Hey, I just made a Pull Request!

Currently we cannot customise empty content, add customised message, component or similar, when there are not entities loaded in Catalog. Think we probably can pass this prop so it is usable in a same way as in other tables,

✔️ 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: irma12 <irma@roadie.io>
@Irma12 Irma12 requested review from a team as code owners October 5, 2022 13:25
@github-actions github-actions bot added the area:catalog Related to the Catalog Project Area label Oct 5, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2022

Changed Packages

Package Name Package Path Changeset Bump Current Version
@backstage/plugin-catalog plugins/catalog patch v1.6.0-next.2

Signed-off-by: irma12 <irma@roadie.io>
@github-actions
Copy link
Contributor

This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If you are the author and the PR has been closed, feel free to re-open the PR and continue the contribution!

@github-actions github-actions bot added the stale label Oct 14, 2022
@Rugvip Rugvip removed the stale label Oct 15, 2022
Signed-off-by: irma12 <irma@roadie.io>
Copy link
Member

@Rugvip Rugvip left a comment

Choose a reason for hiding this comment

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

Thank you, makes sense! 👍

Just a minor concern

Comment on lines 60 to 64
empty: {
padding: theme.spacing(2),
display: 'flex',
justifyContent: 'center',
},
Copy link
Member

Choose a reason for hiding this comment

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

I wonder if this styling is perhaps a bit too opinionated, should we leave it to the content instead? Especially since it's not possible to override

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, agree it is too strict. Removed it :)

Signed-off-by: irma12 <irma@roadie.io>
Signed-off-by: irma12 <irma@roadie.io>
Copy link
Member

@Rugvip Rugvip left a comment

Choose a reason for hiding this comment

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

Great, thank you! :shipit:

@Rugvip Rugvip merged commit e167b94 into backstage:master Oct 17, 2022
@suuus
Copy link
Contributor

suuus commented Oct 18, 2022

Thank you for contributing during Hacktoberfest 🍂! You can claim the Backstage Hacktoberfest Holopin at https://bck.st/hacktoberfest-holopin 🙌🏻

@Irma12 Irma12 mentioned this pull request Apr 10, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:catalog Related to the Catalog Project Area
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants