-
Notifications
You must be signed in to change notification settings - Fork 0
Initial draft adding advanced grouping and filtering #1
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Andre Wanlin <[email protected]>
awanlin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left comments on the parts I could use some help with
| <m.InternalRouter formFieldLoaders={formFieldLoaders} />, | ||
| <m.InternalRouter | ||
| formFieldLoaders={formFieldLoaders} | ||
| groups={config.groups} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do I translate an EntityPredicate to (entity: TemplateEntityV1beta3) => boolean?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
entityPredicateToFilterFunction from @backstage/plugin-catalog-react/alpha 😁
| defaultPath: '/docs', | ||
| loader: () => | ||
| import('../home/components/TechDocsCustomHome').then(m => | ||
| compatWrapper(<m.TechDocsCustomHome tabsConfig={config.tabs} />), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think doing an if somewhere here where we check for config.tabs, if it's provided we use TechDocsCustomHome otherwise we use TechDocsIndexPage. I'm just not sure where that would go.
Also have some type issues here like getting it to be happy with the PanelType and the filterPredicate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep something like that sounds good, might be best to push some WIP code here to have something to look at
| panelType: 'InfoCardGrid' | ||
| filter: | ||
| metadata.tags: | ||
| $in: ['golden-path'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the end goal I'm trying to make work. I want to use the filtering that was introduced here: backstage#28951
Hey, I just made a Pull Request!
✔️ Checklist
Signed-off-byline in the message. (more info)