-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add dynamic-home-page workspace with backstage app and plugin scaffold #28
Add dynamic-home-page workspace with backstage app and plugin scaffold #28
Conversation
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
6ca2150
to
c81e0bd
Compare
4c9dd7d
to
886b901
Compare
886b901
to
2255e00
Compare
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
2255e00
to
31bc3ed
Compare
There are example and template folders with YAML files that the catalog locations use. Do you need these examples for your dynamic homepage? |
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 have verified that the yarn start
and yarn dev
starts the application without any errors
/lgtm
Do you want this merged as is, or are you proposing adding more files to the PR first? |
Merge as it is, improve later. It should not contain examples, it should contain the code that is currently in the showcase repo
(Testing answering on the GitHub email)
|
Done
|
Hey, I just made a Pull Request!
A workspace to extract/migrate https://github.com/janus-idp/backstage-showcase/tree/main/plugins/dynamic-home-page
It contains a workspace with the backstage app and a plugin (frontend scaffold) that is marked as private, so that I can migrate the code in a follow-up PR.
Replacing the bundled plugin in 1.4 would be nice, but that's a stretch goal.
✔️ Checklist