Conversation
There was a problem hiding this comment.
Pull request overview
This pull request updates package names from the @edx scope to the @2u scope for four frontend enterprise packages across the entire codebase. This appears to be part of a broader organizational rebranding effort referenced in ticket GSRE-3458.
Changes:
- Updated package.json to reference
@2uscoped packages instead of@edxfor: frontend-enterprise-catalog-search, frontend-enterprise-hotjar, frontend-enterprise-logistration, and frontend-enterprise-utils - Updated 151+ import statements across all source files to use the new
@2uscoped package names - Updated all jest.mock() calls and test imports to reference the new package names
- Updated comments referencing these packages
Reviewed changes
Copilot reviewed 147 out of 147 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updated dependencies from @edx to @2u scope for 4 enterprise packages |
| src/index.jsx | Updated hasFeatureFlagEnabled import from @2u/frontend-enterprise-utils |
| src/config/index.js | Updated hasFeatureFlagEnabled import from @2u/frontend-enterprise-utils |
| src/components/test/testUtils.jsx | Updated TODO comment reference to @2u/frontend-enterprise-utils |
| src/components/subscriptions/* | Updated sendEnterpriseTrackEvent imports and jest mocks across subscription components |
| src/components/settings/* | Updated sendEnterpriseTrackEvent imports across settings components |
| src/components/learner-credit-management/* | Updated multiple imports including sendEnterpriseTrackEvent, renderWithRouter, and search components from @2u packages |
| src/components/UserActivationPage/index.jsx | Updated LoginRedirect import from @2u/frontend-enterprise-logistration |
| src/components/TableComponent/* | Updated sendEnterpriseTrackEvent imports and mocks |
| src/components/ProductTours/* | Updated sendEnterpriseTrackEvent imports across all tour components |
| src/components/PeopleManagement/* | Updated sendEnterpriseTrackEvent and renderWithRouter imports |
| src/components/Header/index.jsx | Updated getProxyLoginUrl import from @2u/frontend-enterprise-logistration |
| src/components/EnterpriseList/* | Updated renderWithRouter import |
| src/components/DownloadCsvButton/index.jsx | Updated sendEnterpriseTrackEvent import |
| src/components/ContentHighlights/* | Updated sendEnterpriseTrackEvent and renderWithRouter imports, plus SearchData/SearchHeader from catalog-search |
| src/components/BulkEnrollmentPage/* | Updated SearchContext, SearchData, SearchHeader imports from @2u/frontend-enterprise-catalog-search |
| src/components/BudgetExpiryAlertAndModal/* | Updated sendEnterpriseTrackEvent and renderWithRouter imports |
| src/components/AuthenticatedEnterpriseApp/index.jsx | Updated LoginRedirect import from @2u/frontend-enterprise-logistration |
| src/components/App/index.jsx | Updated initializeHotjar import from @2u/frontend-enterprise-hotjar |
| src/components/AdvanceAnalyticsV2.0/* | Updated sendEnterpriseTrackEvent imports and mocks |
| src/components/AdminV2/* | Updated sendEnterpriseTrackEvent imports and mocks |
| src/components/AdminRegisterPage/* | Updated LoginRedirect, getProxyLoginUrl, isEnterpriseUser, and ENTERPRISE_ADMIN imports from @2u packages |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 147 out of 148 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/components/learner-credit-management/search/tests/CatalogSearchResults.test.jsx
Show resolved
Hide resolved
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 148 out of 149 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.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 148 out of 150 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/components/learner-credit-management/search/tests/CatalogSearchResults.test.jsx
Outdated
Show resolved
Hide resolved
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #29 +/- ##
==========================================
+ Coverage 85.49% 85.51% +0.02%
==========================================
Files 816 816
Lines 19025 19033 +8
Branches 3911 4015 +104
==========================================
+ Hits 16266 16277 +11
+ Misses 2694 2662 -32
- Partials 65 94 +29 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 151 out of 153 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.
GSRE-3458
update package names from edx to 2U