-
Notifications
You must be signed in to change notification settings - Fork 154
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
chore(wrapper): add wrapper for marketplace frontend, backend and catalog-backend-module plugins #2156
base: main
Are you sure you want to change the base?
chore(wrapper): add wrapper for marketplace frontend, backend and catalog-backend-module plugins #2156
Conversation
62d25cc
to
0c4d778
Compare
/test e2e-tests |
The image is available at: |
d1f2829
to
4235077
Compare
The image is available at: |
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.
Sorry :)
@@ -1010,3 +1010,26 @@ plugins: | |||
catalog: | |||
providers: | |||
pingIdentityOrg: {} | |||
# Group: Marketplace | |||
- package: ./dynamic-plugins/dist/red-hat-developer-hub-backstage-plugin-catalog-backend-module-marketplace-dynamic | |||
disabled: true |
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.
👍
...d-hat-developer-hub-backstage-plugin-catalog-backend-module-marketplace-dynamic/package.json
Outdated
Show resolved
Hide resolved
...ins/wrappers/red-hat-developer-hub-backstage-plugin-marketplace-backend-dynamic/package.json
Outdated
Show resolved
Hide resolved
...ugins/wrappers/red-hat-developer-hub-backstage-plugin-marketplace-backend-dynamic/turbo.json
Outdated
Show resolved
Hide resolved
dynamic-plugins/wrappers/red-hat-developer-hub-backstage-plugin-marketplace/package.json
Outdated
Show resolved
Hide resolved
dynamic-plugins/wrappers/red-hat-developer-hub-backstage-plugin-marketplace/turbo.json
Outdated
Show resolved
Hide resolved
The image is available at: |
}, | ||
"repository": { | ||
"type": "git", | ||
"url": "https://github.com/janus-idp/backstage-showcase", |
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.
Reminder for myself: we need to update this
"directory": "dynamic-plugins/wrappers/red-hat-developer-hub-backstage-plugin-marketplace" | ||
}, | ||
"maintainers": [ | ||
"@janus-idp/maintainers-showcase" |
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.
Reminder for myself: we need to update this
/retest |
2 similar comments
/retest |
/retest |
plugins/dynamic-plugins-info/src/components/InternalPluginsMap.ts
Outdated
Show resolved
Hide resolved
d43de66
to
69e1b75
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The image is available at: |
This lgtm, just waiting for for e2e tests after we migrated this git repo. Will add lgtm label then. |
/retest |
/retest |
Description
Add dynamic plugin wrappers for marketplace, marketplace-backend and catalog-backend-module-marketplace plugins.
Which issue(s) does this PR fix
https://issues.redhat.com/browse/RHIDP-5322
PR acceptance criteria
Please make sure that the following steps are complete:
How to test changes / Special notes to the reviewer