Skip to content

feat: register adoption insights translation resources in dynamic plugins config#4950

Open
lokanandaprabhu wants to merge 2 commits into
redhat-developer:mainfrom
lokanandaprabhu:feat/adoption-insights-translation-resources
Open

feat: register adoption insights translation resources in dynamic plugins config#4950
lokanandaprabhu wants to merge 2 commits into
redhat-developer:mainfrom
lokanandaprabhu:feat/adoption-insights-translation-resources

Conversation

@lokanandaprabhu

Copy link
Copy Markdown
Member

Which issue(s) does this PR fix

Summary

Registers translation resources for the Adoption Insights dynamic plugin in app-config.dynamic-plugins.yaml, enabling i18n support when the plugin is loaded as a dynamic plugin.
Adds the translationResources entry for red-hat-developer-hub.backstage-plugin-adoption-insights:

  • importName: adoptionInsightsTranslations
  • ref: adoptionInsightsTranslationRef
    This follows the same pattern used by other RHDH dynamic plugins (e.g. Extensions, Dynamic Home Page).

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

…gins config

Co-authored-by: Cursor <cursoragent@cursor.com>
@openshift-ci openshift-ci Bot requested review from albarbaro and polasudo June 16, 2026 09:39
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

@codecov

codecov Bot commented Jun 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.82%. Comparing base (0125ff6) to head (c8b99f5).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4950   +/-   ##
=======================================
  Coverage   55.82%   55.82%           
=======================================
  Files         121      121           
  Lines        2350     2350           
  Branches      539      539           
=======================================
  Hits         1312     1312           
  Misses       1033     1033           
  Partials        5        5           
Flag Coverage Δ
rhdh 55.82% <ø> (ø)

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0125ff6...c8b99f5. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Add the missing Alpha wrapper export so dynamic plugin translation
resources can resolve adoptionInsightsTranslations at runtime.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

@openshift-ci

openshift-ci Bot commented Jun 16, 2026

Copy link
Copy Markdown

@lokanandaprabhu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-ocp-helm c8b99f5 link true /test e2e-ocp-helm

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

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.

1 participant