Skip to content

[FEAT] [UI] Internalize igz-controls from dashboard-react-controls#3713

Open
arturrybalskyy-dev wants to merge 1 commit into
mlrun:developmentfrom
arturrybalskyy-dev:feat/ML-12800/internalize-igz-controls
Open

[FEAT] [UI] Internalize igz-controls from dashboard-react-controls#3713
arturrybalskyy-dev wants to merge 1 commit into
mlrun:developmentfrom
arturrybalskyy-dev:feat/ML-12800/internalize-igz-controls

Conversation

@arturrybalskyy-dev

Copy link
Copy Markdown
Contributor

📝 Description

Moves all used dashboard-react-controls (DRC) components, hooks, utils, constants, types, styles, and images into ui/src/igz-controls/ and redirects the existing igz-controls Vite alias from node_modules to the local path.

This removes the iguazio.dashboard-react-controls npm dependency entirely. All 391 source files that import from igz-controls/... require zero changes - the alias preservation strategy ensures full backwards compatibility.


🛠️ Changes Made

  • Added ui/src/igz-controls/ - full local copy of all used DRC entities (components, hooks, utils, constants, types, scss, fonts, images, nextGenComponents)
  • Updated vite.config.mjs and vitest.config.mjs - redirected igz-controls alias from node_modules/iguazio.dashboard-react-controls/dist to src/igz-controls/
  • Updated jsconfig.json and tsconfig.json - added matching path mappings for IDE resolution
  • Updated package.json - removed iguazio.dashboard-react-controls dependency; removed nli/nui scripts that installed/updated DRC
  • Updated eslint.config.mjs - added src/igz-controls/nextGenComponents/** to ignores (TypeScript files requiring TS-aware parser)
  • Updated .github/workflows/ci.yaml - updated error hint to reference ui/src/igz-controls/ instead of the removed npm package
  • Added src/igz-controls/index.scss —-replaces the former dist/index.css global stylesheet bundle

✅ Checklist

  • I have given the PR a well-structured title describing the domain and the specific change that was made
  • I tested the changes in the browser (locally or via preview build)
  • I confirmed that existing tests pass
  • I added or updated unit / integration tests (if needed)
  • I checked that this change doesn't introduce new console warnings or lint / formatting errors
  • I updated the relevant Jira ticket with the appropriate details and status

🔗 References

  • Related ticket / issue: ML-12800

🚨 Potentially Breaking Changes

  • Yes
  • No

Includes DRC change

  • Yes
  • No

🔍 Additional Notes

  • Unused entities not moved: ChipForm component and proxyServerConfig util were intentionally excluded per task requirement ("Do not move unused entities")
  • Images: 43 unused images were identified and excluded from the migration

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