Skip to content

Fix Cost Management Dynamic Plugin Routes#2462

Merged
asmasarw merged 1 commit intoredhat-developer:mainfrom
asmasarw:fix/cost-management-dynamic-plugin
Mar 5, 2026
Merged

Fix Cost Management Dynamic Plugin Routes#2462
asmasarw merged 1 commit intoredhat-developer:mainfrom
asmasarw:fix/cost-management-dynamic-plugin

Conversation

@asmasarw
Copy link
Contributor

@asmasarw asmasarw commented Mar 5, 2026

Fix Dynamic Plugin Configuration to show menu items in the sidebar.

@rhdh-qodo-merge
Copy link

Review Summary by Qodo

Fix Cost Management Plugin Menu Item Identifiers

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Fix menu item identifiers using forward slashes instead of dots
• Update cost-management plugin route references in configuration files
• Ensure consistent naming convention across dynamic plugin configurations
Diagram
flowchart LR
  A["Menu Item Keys<br/>with dots"] -- "Replace dots<br/>with slashes" --> B["Menu Item Keys<br/>with slashes"]
  C["dynamic-plugins.yaml"] -- "Update identifiers" --> B
  D["app-config.dynamic.yaml"] -- "Update identifiers" --> B
Loading

Grey Divider

File Changes

1. workspaces/cost-management/dynamic-plugins.yaml 🐞 Bug fix +2/-2

Update menu item identifiers with forward slashes

• Changed menu item identifier cost-management.optimizations to cost-management/optimizations
• Changed menu item identifier cost-management.openshift to cost-management/openshift
• Maintains parent and priority configuration values

workspaces/cost-management/dynamic-plugins.yaml


2. workspaces/cost-management/plugins/cost-management/app-config.dynamic.yaml 🐞 Bug fix +2/-2

Update menu item identifiers with forward slashes

• Changed menu item identifier cost-management.optimizations to cost-management/optimizations
• Changed menu item identifier cost-management.openshift to cost-management/openshift
• Maintains parent and priority configuration values

workspaces/cost-management/plugins/cost-management/app-config.dynamic.yaml


Grey Divider

Qodo Logo

@rhdh-qodo-merge
Copy link

rhdh-qodo-merge bot commented Mar 5, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 5, 2026

@rhdh-gh-app
Copy link

rhdh-gh-app bot commented Mar 5, 2026

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @red-hat-developer-hub/plugin-cost-management

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/plugin-cost-management workspaces/cost-management/plugins/cost-management none v2.0.1

@asmasarw asmasarw enabled auto-merge (squash) March 5, 2026 09:52
@asmasarw asmasarw merged commit 7a2cd40 into redhat-developer:main Mar 5, 2026
9 checks passed
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