Skip to content

fix(scaffolder): fix template card type in entity-scaffolder-content#70

Open
roope-t wants to merge 1 commit into
seatgeek:mainfrom
roope-t:bugfix/template-card-type
Open

fix(scaffolder): fix template card type in entity-scaffolder-content#70
roope-t wants to merge 1 commit into
seatgeek:mainfrom
roope-t:bugfix/template-card-type

Conversation

@roope-t
Copy link
Copy Markdown
Contributor

@roope-t roope-t commented Sep 25, 2024

Fixes the type of TemplateCard prop in EntityScaffolderContent component.

@roope-t
Copy link
Copy Markdown
Contributor Author

roope-t commented Sep 25, 2024

Hey @zhammer, this fixes a small issue that I noticed now when starting to use the plugin.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects the typing of the TemplateCard override in EntityScaffolderContent so that consumers must provide a component compatible with the TemplateGroups API from @backstage/plugin-scaffolder-react/alpha.

Changes:

  • Import TemplateCard from @backstage/plugin-scaffolder-react/alpha.
  • Update EntityScaffolderContentProps.components.TemplateCard to use typeof TemplateCard instead of a custom React.ComponentType signature.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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