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
CONSOLE-2904: Allow disabling dynamic plugins via query parameter #9363
CONSOLE-2904: Allow disabling dynamic plugins via query parameter #9363
Conversation
0e17237
to
0fcd6fb
Compare
|
||
if (process.env.NODE_ENV !== 'production') { | ||
// Expose Console plugin store for debugging | ||
window.pluginStore = pluginStore; | ||
|
||
// Expose function to load dynamic plugins directly from URLs | ||
window.loadPluginFromURL = loadPluginFromURL; |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
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.
LGTM. We should update the dynamic plugins enhancement for the change to the query param (using ""
instead of all
to match all plugins).
0fcd6fb
to
5b22935
Compare
Rebased and modified |
@vojtechszocs #8919 has merged, so you should be able to rebase |
5b22935
to
0fd8c59
Compare
Rebased and reflected the change in openshift/enhancements#820 |
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.
/approve
/assign @yapei @ahardin-rh @RickJWagner
for approvals
/hold
frontend/public/plugins.ts
Outdated
import { loadPluginFromURL } from '@console/dynamic-plugin-sdk/src/runtime/plugin-loader'; | ||
import { getURLSearchParams } from './components/utils/link'; | ||
|
||
const getAllowedDynamicPluginNames = () => { |
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.
nit: Should this be getEnabledDynamicPluginNames
?
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.
Sounds good, will update.
/label docs-approved |
0fd8c59
to
234a4f7
Compare
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: spadgett, vojtechszocs The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
1 similar comment
/retest |
tested(detailed steps are in test case) and no issues found |
/unassign @RickJWagner @sferich888 This lets users disable any dynamic plugin using a query parameter |
/label px-approved |
/hold cancel |
This PR implements the
disable-plugins
query parameter as outlined in "Error Handling" section of OpenShift Console Dynamic Plugins feature page.?disable-plugins
or?disable-plugins=
prevents loading of any dynamic plugins (disable all)?disable-plugins=foo,bar
prevents loading of dynamic plugins namedfoo
orbar
(disable selectively)This has no effect on static plugins, which are built into the Console application.