🌐(lib_components) add the language in the request header

The language sent to the backend was the browser one, we now send the the language setted in the app to get the backend messages internationalized.
openfun · Sep 4, 2023 · 8ffa619 · 8ffa619
1 parent f767f1b
commit 8ffa619
Show file tree

Hide file tree

Showing 103 changed files with 339 additions and 7 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,10 @@ Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
 
+### Added
+
+- Add the language to header of most of the http request (#2394)
+
 ## [4.3.1] - 2023-08-31
 
 ### Fixed

diff --git a/...ontend/apps/lti_site/apps/deposit/components/Dashboard/DashboardInstructor/index.spec.tsx b/...ontend/apps/lti_site/apps/deposit/components/Dashboard/DashboardInstructor/index.spec.tsx
@@ -294,6 +294,7 @@ describe('<DashboardInstructor />', () => {
       headers: {
         Authorization: 'Bearer json web token',
         'Content-Type': 'application/json',
+        'Accept-Language': 'en',
       },
       method: 'PATCH',
       body: JSON.stringify({ title: 'My file depository title updated' }),
@@ -346,6 +347,7 @@ describe('<DashboardInstructor />', () => {
       headers: {
         Authorization: 'Bearer json web token',
         'Content-Type': 'application/json',
+        'Accept-Language': 'en',
       },
       method: 'PATCH',
       body: JSON.stringify({
@@ -401,6 +403,7 @@ describe('<DashboardInstructor />', () => {
       headers: {
         Authorization: 'Bearer json web token',
         'Content-Type': 'application/json',
+        'Accept-Language': 'en',
       },
       method: 'PATCH',
       body: JSON.stringify({
@@ -453,6 +456,7 @@ describe('<DashboardInstructor />', () => {
       headers: {
         Authorization: 'Bearer json web token',
         'Content-Type': 'application/json',
+        'Accept-Language': 'en',
       },
       method: 'PATCH',
       body: JSON.stringify({

diff --git a/...nd/apps/lti_site/apps/deposit/components/Dashboard/common/DepositedFileRow/index.spec.tsx b/...nd/apps/lti_site/apps/deposit/components/Dashboard/common/DepositedFileRow/index.spec.tsx
@@ -56,6 +56,7 @@ describe('<DepositedFileRow />', () => {
       headers: {
         Authorization: 'Bearer json web token',
         'Content-Type': 'application/json',
+        'Accept-Language': 'en',
       },
       method: 'PATCH',
       body: JSON.stringify({ read: true }),

diff --git a/src/frontend/apps/lti_site/apps/deposit/data/queries/index.spec.tsx b/src/frontend/apps/lti_site/apps/deposit/data/queries/index.spec.tsx
@@ -57,6 +57,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(fileDepositories);
@@ -82,6 +83,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(undefined);
@@ -115,6 +117,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(fileDepository);
@@ -143,6 +146,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(undefined);
@@ -172,6 +176,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({
@@ -204,6 +209,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({
@@ -245,6 +251,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'PATCH',
         body: JSON.stringify({
@@ -279,6 +286,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'PATCH',
         body: JSON.stringify({
@@ -319,6 +327,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(depositedFiles.slice(0, 3));
@@ -353,6 +362,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(depositedFiles.slice(3, 4));
@@ -387,6 +397,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(undefined);
@@ -426,6 +437,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'PATCH',
         body: JSON.stringify({
@@ -467,6 +479,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'PATCH',
         body: JSON.stringify({

diff --git a/src/frontend/apps/lti_site/apps/deposit/data/sideEffects/createDepositedFile/index.spec.ts b/src/frontend/apps/lti_site/apps/deposit/data/sideEffects/createDepositedFile/index.spec.ts
@@ -46,6 +46,7 @@ describe('sideEffects/createDepositedFile', () => {
     expect(fetchArgs.headers).toEqual({
       Authorization: 'Bearer token',
       'Content-Type': 'application/json',
+      'Accept-Language': 'en',
     });
     expect(fetchArgs.method).toEqual('POST');
   });

diff --git a/src/frontend/apps/lti_site/components/DashboardDocument/index.spec.tsx b/src/frontend/apps/lti_site/components/DashboardDocument/index.spec.tsx
@@ -67,6 +67,7 @@ describe('<DashboardDocument />', () => {
     );
     expect(fetchMock.lastCall()![1]!.headers).toEqual({
       Authorization: 'Bearer cool_token_m8',
+      'Accept-Language': 'en',
     });
     screen.getByText('Processing');
 

diff --git a/src/frontend/apps/lti_site/components/PlaylistPortability/index.spec.tsx b/src/frontend/apps/lti_site/components/PlaylistPortability/index.spec.tsx
@@ -64,6 +64,7 @@ describe('<PlaylistPortability />', () => {
       headers: {
         Authorization: 'Bearer some token',
         'Content-Type': 'application/json',
+        'Accept-Language': 'en',
       },
     });
 
@@ -101,6 +102,7 @@ describe('<PlaylistPortability />', () => {
       headers: {
         Authorization: 'Bearer some token',
         'Content-Type': 'application/json',
+        'Accept-Language': 'en',
       },
       method: 'PATCH',
       body: JSON.stringify({
@@ -117,6 +119,7 @@ describe('<PlaylistPortability />', () => {
       headers: {
         Authorization: 'Bearer some token',
         'Content-Type': 'application/json',
+        'Accept-Language': 'en',
       },
     });
 

diff --git a/src/frontend/apps/lti_site/data/queries/index.spec.tsx b/src/frontend/apps/lti_site/data/queries/index.spec.tsx
@@ -55,6 +55,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(organization);
@@ -80,6 +81,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toBeUndefined();
@@ -106,6 +108,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(playlist);
@@ -131,6 +134,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(undefined);
@@ -160,6 +164,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'PATCH',
         body: JSON.stringify({
@@ -191,6 +196,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'PATCH',
         body: JSON.stringify({
@@ -221,6 +227,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(playlists);
@@ -245,6 +252,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(undefined);
@@ -278,6 +286,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(thumbnail);
@@ -309,6 +318,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
       });
       expect(result.current.data).toEqual(undefined);
@@ -337,6 +347,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({
@@ -370,6 +381,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({
@@ -405,6 +417,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({
@@ -441,6 +454,7 @@ describe('queries', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({

diff --git a/src/frontend/apps/lti_site/data/sideEffects/deleteTimedTextTrack/index.spec.ts b/src/frontend/apps/lti_site/data/sideEffects/deleteTimedTextTrack/index.spec.ts
@@ -18,6 +18,7 @@ describe('sideEffects/deleteTimedTextTrack', () => {
 
     expect(fetchMock.lastCall()![1]!.headers).toEqual({
       Authorization: 'Bearer some token',
+      'Accept-Language': 'en',
     });
     expect(response).toBe(true);
   });

diff --git a/src/frontend/apps/standalone_site/src/api/useConfig/index.spec.tsx b/src/frontend/apps/standalone_site/src/api/useConfig/index.spec.tsx
@@ -30,6 +30,7 @@ describe('useConfig', () => {
     expect(fetchMock.lastCall()![1]).toEqual({
       headers: {
         'Content-Type': 'application/json',
+        'Accept-Language': 'en',
       },
     });
     expect(result.current.data).toEqual({
@@ -55,6 +56,7 @@ describe('useConfig', () => {
     expect(fetchMock.lastCall()![0]).toEqual('/api/config/');
     expect(fetchMock.lastCall()![1]).toEqual({
       headers: {
+        'Accept-Language': 'en',
         'Content-Type': 'application/json',
       },
     });

diff --git a/src/frontend/apps/standalone_site/src/api/useLtiUserAssociations/index.spec.tsx b/src/frontend/apps/standalone_site/src/api/useLtiUserAssociations/index.spec.tsx
@@ -35,6 +35,7 @@ describe('features/PortabilityRequests/api/useLtiUserAssociations', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({
@@ -65,6 +66,7 @@ describe('features/PortabilityRequests/api/useLtiUserAssociations', () => {
         headers: {
           Authorization: 'Bearer some token',
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({

diff --git a/src/frontend/apps/standalone_site/src/features/Authentication/api/basicLogin/index.spec.tsx b/src/frontend/apps/standalone_site/src/features/Authentication/api/basicLogin/index.spec.tsx
@@ -33,6 +33,7 @@ describe('features/Authentication/api/basicLogin', () => {
       expect(fetchMock.lastCall()![1]).toEqual({
         headers: {
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({
@@ -70,6 +71,7 @@ describe('features/Authentication/api/basicLogin', () => {
       expect(fetchMock.lastCall()![1]).toEqual({
         headers: {
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({

diff --git a/...tend/apps/standalone_site/src/features/Authentication/api/usePasswordReset/index.spec.tsx b/...tend/apps/standalone_site/src/features/Authentication/api/usePasswordReset/index.spec.tsx
@@ -31,6 +31,7 @@ describe('features/Authentication/api/usePasswordReset', () => {
       expect(fetchMock.lastCall()![1]).toEqual({
         headers: {
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({
@@ -64,6 +65,7 @@ describe('features/Authentication/api/usePasswordReset', () => {
       expect(fetchMock.lastCall()![1]).toEqual({
         headers: {
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({

diff --git a/...ps/standalone_site/src/features/Authentication/api/usePasswordResetConfirm/index.spec.tsx b/...ps/standalone_site/src/features/Authentication/api/usePasswordResetConfirm/index.spec.tsx
@@ -35,6 +35,7 @@ describe('features/Authentication/api/usePasswordResetConfirm', () => {
       expect(fetchMock.lastCall()![1]).toEqual({
         headers: {
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({
@@ -74,6 +75,7 @@ describe('features/Authentication/api/usePasswordResetConfirm', () => {
       expect(fetchMock.lastCall()![1]).toEqual({
         headers: {
           'Content-Type': 'application/json',
+          'Accept-Language': 'en',
         },
         method: 'POST',
         body: JSON.stringify({