-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Added i18n for search header component. #1084
Conversation
9777592
to
c64cbc7
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1084 +/- ##
==========================================
+ Coverage 86.00% 86.01% +0.01%
==========================================
Files 381 381
Lines 7759 7774 +15
Branches 1893 1894 +1
==========================================
+ Hits 6673 6687 +14
- Misses 1032 1033 +1
Partials 54 54 ☔ View full report in Codecov by Sentry. |
e650568
to
126e423
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with a minor nit.
package.json
Outdated
@@ -9,7 +9,7 @@ | |||
"dependencies": { | |||
"@edx/brand": "npm:@openedx/brand-openedx@1.2.2", | |||
"@edx/frontend-component-footer": "13.0.2", | |||
"@edx/frontend-enterprise-catalog-search": "9.0.0", | |||
"@edx/frontend-enterprise-catalog-search": "^10.3.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: pin the dependency (similar to others; maintain convention) and re-run npm install
to update package-lock.json.
126e423
to
160699a
Compare
Jira Ticket: ENT-8580
Description:
This PR adds i18n support for search header component.
For all changes
Only if submitting a visual change