Conversation
There was a problem hiding this comment.
Pull request overview
This pull request attempts to migrate four npm packages from the @edx scope to the @2uinc scope while maintaining the same version numbers. The PR description indicates this is related to ticket GSRE-3458 and is described as installing new packages.
Changes:
- Updates package scope from
@edxto@2uincfor four enterprise-related npm packages - Maintains existing version numbers for all affected packages
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "@2uinc/frontend-enterprise-catalog-search": "11.0.1", | ||
| "@2uinc/frontend-enterprise-hotjar": "7.2.0", | ||
| "@2uinc/frontend-enterprise-logistration": "9.2.1", | ||
| "@2uinc/frontend-enterprise-utils": "10.0.0", |
There was a problem hiding this comment.
The package scope has been changed from @edx to @2uinc in package.json, but all import statements throughout the codebase still reference the old @edx scoped packages. This will cause runtime errors when the application tries to import these modules.
The following files have imports that need to be updated:
- src/components/BulkEnrollmentPage/CourseSearchResults.jsx
- src/components/BulkEnrollmentPage/CourseSearchResults.test.jsx
- src/components/BulkEnrollmentPage/stepper/AddCoursesStep.tsx
- src/components/ContentHighlights/HighlightStepper/HighlightStepperSelectContentSearch.tsx
- src/components/learner-credit-management/BudgetDetailCatalogTabContents.jsx
- src/components/learner-credit-management/search/CatalogSearch.tsx
- src/components/learner-credit-management/search/CatalogSearchResults.jsx
- src/components/learner-credit-management/search/tests/CatalogSearch.test.tsx
- src/components/learner-credit-management/search/tests/CatalogSearchResults.test.jsx
- src/components/App/index.jsx (for hotjar)
- src/components/AdminRegisterPage/index.jsx (for logistration)
- src/components/AuthenticatedEnterpriseApp/index.jsx (for logistration)
- src/components/Header/index.jsx (for logistration)
- src/components/UserActivationPage/index.jsx (for logistration)
- And many more files importing from
@edx/frontend-enterprise-utils
All import statements need to be updated to use @2uinc instead of @edx for these four packages.
| "@2uinc/frontend-enterprise-catalog-search": "11.0.1", | |
| "@2uinc/frontend-enterprise-hotjar": "7.2.0", | |
| "@2uinc/frontend-enterprise-logistration": "9.2.1", | |
| "@2uinc/frontend-enterprise-utils": "10.0.0", | |
| "@edx/frontend-enterprise-catalog-search": "npm:@2uinc/frontend-enterprise-catalog-search@11.0.1", | |
| "@edx/frontend-enterprise-hotjar": "npm:@2uinc/frontend-enterprise-hotjar@7.2.0", | |
| "@edx/frontend-enterprise-logistration": "npm:@2uinc/frontend-enterprise-logistration@9.2.1", | |
| "@edx/frontend-enterprise-utils": "npm:@2uinc/frontend-enterprise-utils@10.0.0", |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
1 participant
GSRE-3458
Install new packages