Skip to content

chore(bulk-import): promote bulk-import from alpha to stable - #3716

Merged
divyanshiGupta merged 7 commits into
redhat-developer:mainfrom
divyanshiGupta:nfs-stable-bulk-import
Jul 24, 2026
Merged

chore(bulk-import): promote bulk-import from alpha to stable#3716
divyanshiGupta merged 7 commits into
redhat-developer:mainfrom
divyanshiGupta:nfs-stable-bulk-import

Conversation

@divyanshiGupta

@divyanshiGupta divyanshiGupta commented Jul 9, 2026

Copy link
Copy Markdown
Member

Hey, I just made a Pull Request!

https://redhat.atlassian.net/browse/RHIDP-14360

Graduated the New Frontend System (NFS) plugin to the primary entry point.

The main . export now only contains NFS APIs (createFrontendPlugin, bulkImportTranslationsModule, translations). Legacy (OFS) component re-exports (bulkImportPlugin, BulkImportPage, BulkImportSidebarItem, BulkImportIcon) have been moved to the ./legacy subpath.

Screen.Recording.2026-07-21.at.11.23.57.AM.mov

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

@rhdh-gh-app

rhdh-gh-app Bot commented Jul 9, 2026

Copy link
Copy Markdown

Important

This PR includes changes that affect public-facing API. Please ensure you are adding/updating documentation for new features or behavior.

Changed Packages

Package Name Package Path Changeset Bump Current Version
app-legacy workspaces/bulk-import/packages/app-legacy none v0.0.3
app workspaces/bulk-import/packages/app none v0.0.1
@red-hat-developer-hub/backstage-plugin-bulk-import workspaces/bulk-import/plugins/bulk-import major v7.3.7

@divyanshiGupta
divyanshiGupta force-pushed the nfs-stable-bulk-import branch 3 times, most recently from e2823cb to a543d6b Compare July 14, 2026 13:21
@divyanshiGupta
divyanshiGupta requested review from a team as code owners July 14, 2026 13:21
@divyanshiGupta
divyanshiGupta force-pushed the nfs-stable-bulk-import branch from a543d6b to 2a07989 Compare July 14, 2026 13:47
@divyanshiGupta
divyanshiGupta force-pushed the nfs-stable-bulk-import branch 11 times, most recently from 20026e1 to 5fbc691 Compare July 20, 2026 11:23
@divyanshiGupta

Copy link
Copy Markdown
Member Author

There is a issue I am facing wherein the CI expects the api-report in a certain format and if formatted using prettier gives api report error. On the otherhand it also shows prettier error if api-report is in the same format it expects.

Run yarn tsc:full && yarn build:all && yarn build:api-reports:only, and you will see the api report is updated

Yeah tried that multiple times. But it is stuck in that api reports/prettier error cycle. Will try again.

@debsmita1

Copy link
Copy Markdown
Member

There is a issue I am facing wherein the CI expects the api-report in a certain format and if formatted using prettier gives api report error. On the otherhand it also shows prettier error if api-report is in the same format it expects.

Run yarn tsc:full && yarn build:all && yarn build:api-reports:only, and you will see the api report is updated

Yeah tried that multiple times. But it is stuck in that api reports/prettier error cycle. Will try again.

I tried it locally with your changes, it updated the report-legacy.api.md file

@debsmita1 debsmita1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested this in RHDH with the help of redhat-developer/rhdh#5115

The icon in the light theme needs to be fixed

Image

@divyanshiGupta

Copy link
Copy Markdown
Member Author

There is a issue I am facing wherein the CI expects the api-report in a certain format and if formatted using prettier gives api report error. On the otherhand it also shows prettier error if api-report is in the same format it expects.

Run yarn tsc:full && yarn build:all && yarn build:api-reports:only, and you will see the api report is updated

Yeah tried that multiple times. But it is stuck in that api reports/prettier error cycle. Will try again.

I tried it locally with your changes, it updated the report-legacy.api.md file

yeah it does. the issue was ci prettier is not accepting what ci api-reports builder requires. anyways i have added these files in .prettierignore so ci shouldnt prettier on these files anymore.

@divyanshiGupta
divyanshiGupta force-pushed the nfs-stable-bulk-import branch from ab25f16 to 68b19c8 Compare July 21, 2026 12:32
@divyanshiGupta

divyanshiGupta commented Jul 21, 2026

Copy link
Copy Markdown
Member Author

There is a issue I am facing wherein the CI expects the api-report in a certain format and if formatted using prettier gives api report error. On the otherhand it also shows prettier error if api-report is in the same format it expects.

Run yarn tsc:full && yarn build:all && yarn build:api-reports:only, and you will see the api report is updated

Yeah tried that multiple times. But it is stuck in that api reports/prettier error cycle. Will try again.

I tried it locally with your changes, it updated the report-legacy.api.md file

yeah it does. the issue was ci prettier is not accepting what ci api-reports builder requires. anyways i have added these files in .prettierignore so ci shouldnt prettier on these files anymore.

@debsmita1 I am talking about this https://github.com/redhat-developer/rhdh-plugins/actions/runs/29830489936/job/88633851082?pr=3716. If I now format it then the ci api-reports builder will give error. Not sure why prettier is still running on the api-reports as they are now in .pettierignore though.

@divyanshiGupta

divyanshiGupta commented Jul 21, 2026

Copy link
Copy Markdown
Member Author

The icon in the light theme needs to be fixed

@debsmita1 is the issue only in this PR or main as well? Since this PR just promotes the ./alpha subpath to the primary . entry point so shouldnt really affect the icon theme colors. Also I dont see this issue while running in the plugin workspace.

@debsmita1

Copy link
Copy Markdown
Member

The icon in the light theme needs to be fixed

@debsmita1 is the issue only in this PR? Since this PR just promotes the ./alpha subpath to the primary . entry point so shouldnt really affect the icon theme colors. Also I dont see this issue while running in the plugin workspace.

I haven't been able to start the rhdh with the bulk-import backend as I am running into errors. But yea, any UI issues can be addressed in a separate ticket

@divyanshiGupta
divyanshiGupta force-pushed the nfs-stable-bulk-import branch 4 times, most recently from b70ac9d to defce8f Compare July 23, 2026 11:26
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.34%. Comparing base (9a3c742) to head (f611851).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3716      +/-   ##
==========================================
- Coverage   57.34%   57.34%   -0.01%     
==========================================
  Files        2381     2382       +1     
  Lines       95582    95586       +4     
  Branches    26694    26697       +3     
==========================================
  Hits        54811    54811              
- Misses      39312    39316       +4     
  Partials     1459     1459              
Flag Coverage Δ *Carryforward flag
adoption-insights 84.54% <ø> (ø) Carriedforward from 9a3c742
ai-integrations 69.06% <ø> (ø) Carriedforward from 9a3c742
app-defaults 69.79% <ø> (ø) Carriedforward from 9a3c742
augment 46.67% <ø> (ø) Carriedforward from 9a3c742
boost 75.83% <ø> (ø) Carriedforward from 9a3c742
bulk-import 72.55% <ø> (-0.06%) ⬇️
cost-management 13.55% <ø> (ø) Carriedforward from 9a3c742
dcm 60.72% <ø> (ø) Carriedforward from 9a3c742
extensions 56.29% <ø> (ø) Carriedforward from 9a3c742
global-floating-action-button 71.18% <ø> (ø) Carriedforward from 9a3c742
global-header 62.17% <ø> (ø) Carriedforward from 9a3c742
homepage 47.70% <ø> (ø) Carriedforward from 9a3c742
install-dynamic-plugins 56.77% <ø> (ø) Carriedforward from 9a3c742
intelligent-assistant 74.05% <ø> (ø) Carriedforward from 9a3c742
konflux 91.98% <ø> (ø) Carriedforward from 9a3c742
lightspeed 69.02% <ø> (ø) Carriedforward from 9a3c742
mcp-integrations 83.40% <ø> (ø) Carriedforward from 9a3c742
orchestrator 62.67% <ø> (ø) Carriedforward from 9a3c742
quickstart 65.04% <ø> (ø) Carriedforward from 9a3c742
sandbox 79.56% <ø> (ø) Carriedforward from 9a3c742
scorecard 82.63% <ø> (ø) Carriedforward from 9a3c742
theme 83.85% <ø> (ø) Carriedforward from 9a3c742
translations 5.12% <ø> (ø) Carriedforward from 9a3c742
x2a 79.31% <ø> (ø) Carriedforward from 9a3c742

*This pull request uses carry forward flags. Click here to find out more.


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 9a3c742...f611851. 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.

@divyanshiGupta
divyanshiGupta force-pushed the nfs-stable-bulk-import branch from defce8f to 96e4dd0 Compare July 23, 2026 13:33

@debsmita1 debsmita1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@divyanshiGupta
divyanshiGupta force-pushed the nfs-stable-bulk-import branch from 96e4dd0 to f611851 Compare July 24, 2026 08:47
@openshift-ci openshift-ci Bot removed the lgtm label Jul 24, 2026
@openshift-ci

openshift-ci Bot commented Jul 24, 2026

Copy link
Copy Markdown

New changes are detected. LGTM label has been removed.

@sonarqubecloud

Copy link
Copy Markdown

@divyanshiGupta
divyanshiGupta enabled auto-merge (squash) July 24, 2026 10:46
@divyanshiGupta
divyanshiGupta merged commit 6e61d04 into redhat-developer:main Jul 24, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants