chore: [UPM-777]/evgeniy/link hreflang and canonical tags seo (deriv-…

…com#14118)
niloofar-deriv · Apr 24, 2024 · 8382cee · 8382cee
1 parent a08a415
commit 8382cee
Show file tree

Hide file tree

Showing 9 changed files with 7 additions and 99 deletions.
diff --git a/packages/account/src/Components/Routes/route-with-sub-routes.tsx b/packages/account/src/Components/Routes/route-with-sub-routes.tsx
@@ -1,15 +1,7 @@
 import React from 'react';
 import { RouteComponentProps } from 'react-router';
 import { Redirect, Route } from 'react-router-dom';
-import {
-    alternateLinkTagChange,
-    canonicalLinkTagChange,
-    redirectToLogin,
-    isEmptyObject,
-    routes,
-    removeBranchName,
-    default_title,
-} from '@deriv/shared';
+import { redirectToLogin, isEmptyObject, routes, removeBranchName, default_title } from '@deriv/shared';
 import { getLanguage } from '@deriv/translations';
 import { TBinaryRoutes, TRoute, TRouteConfig } from '../../Types';
 
@@ -54,9 +46,6 @@ const RouteWithSubRoutes = (route: TRouteWithSubRoutesProps) => {
         const title = route.getTitle?.() ?? '';
         document.title = `${title} | ${default_title}`;
 
-        alternateLinkTagChange();
-        canonicalLinkTagChange();
-
         return result;
     };
 

diff --git a/packages/account/src/Sections/Verification/ProofOfIdentity/proof-of-identity.jsx b/packages/account/src/Sections/Verification/ProofOfIdentity/proof-of-identity.jsx
@@ -1,24 +1,9 @@
-import { AutoHeightWrapper } from '@deriv/components';
-import ProofOfIdentityContainer from './proof-of-identity-container.jsx';
 import React from 'react';
-import { changeMetaTagWithOG } from '@deriv/shared';
 import { withRouter } from 'react-router-dom';
+import { AutoHeightWrapper } from '@deriv/components';
+import ProofOfIdentityContainer from './proof-of-identity-container.jsx';
 
 const ProofOfIdentity = ({ is_from_external, onStateChange }) => {
-    // next useEffect implements seo requirements
-    React.useEffect(() => {
-        const description_content = 'Submit your proof of identity documents to verify your account and start trading';
-        const title_content = 'Account Verification | Deriv app';
-
-        const restoreMetaTagWithOGDescription = changeMetaTagWithOG('description', description_content);
-        const restoreMetaTagWithOGTitle = changeMetaTagWithOG('title', title_content);
-
-        return () => {
-            restoreMetaTagWithOGDescription();
-            restoreMetaTagWithOGTitle();
-        };
-    }, []);
-
     return (
         <AutoHeightWrapper default_height={200}>
             {({ setRef, height }) => (

diff --git a/packages/cashier/src/containers/routes/route-with-sub-routes.tsx b/packages/cashier/src/containers/routes/route-with-sub-routes.tsx
@@ -1,14 +1,7 @@
 import React from 'react';
 import { RouteComponentProps } from 'react-router';
 import { Redirect, Route } from 'react-router-dom';
-import {
-    alternateLinkTagChange,
-    canonicalLinkTagChange,
-    redirectToLogin,
-    routes,
-    removeBranchName,
-    default_title,
-} from '@deriv/shared';
+import { redirectToLogin, routes, removeBranchName, default_title } from '@deriv/shared';
 import { getLanguage } from '@deriv/translations';
 import { TRootStore, TRouteConfig, TRoute } from '../../types';
 
@@ -47,9 +40,6 @@ const RouteWithSubRoutes = (route: TRouteWithSubRoutesProps) => {
         const title = route.getTitle?.() || '';
         document.title = `${title} | ${default_title}`;
 
-        alternateLinkTagChange();
-        canonicalLinkTagChange();
-
         return result;
     };
 

diff --git a/packages/components/src/components/route-with-subroutes/route-with-subroutes.tsx b/packages/components/src/components/route-with-subroutes/route-with-subroutes.tsx
@@ -1,8 +1,6 @@
 import React from 'react';
 import { Redirect, RedirectProps, Route, RouteComponentProps, RouteProps } from 'react-router-dom';
 import {
-    alternateLinkTagChange,
-    canonicalLinkTagChange,
     redirectToLogin,
     removeBranchName,
     routes as shared_routes,
@@ -93,9 +91,6 @@ const RouteWithSubRoutes = ({
         const title = getTitle?.() || '';
         document.title = `${title} | ${default_title}`;
 
-        alternateLinkTagChange();
-        canonicalLinkTagChange();
-
         return result;
     };
 

diff --git a/packages/core/src/App/Components/Routes/route-with-sub-routes.jsx b/packages/core/src/App/Components/Routes/route-with-sub-routes.jsx
@@ -1,14 +1,6 @@
 import React from 'react';
 import { Redirect, Route } from 'react-router-dom';
-import {
-    alternateLinkTagChange,
-    canonicalLinkTagChange,
-    redirectToLogin,
-    removeBranchName,
-    routes,
-    isEmptyObject,
-    default_title,
-} from '@deriv/shared';
+import { redirectToLogin, removeBranchName, routes, isEmptyObject, default_title } from '@deriv/shared';
 import { getLanguage } from '@deriv/translations';
 import Page404 from 'Modules/Page404';
 import { observer, useStore } from '@deriv/stores';
@@ -88,9 +80,6 @@ const RouteWithSubRoutes = observer(route => {
         const title = route.getTitle?.() || '';
         document.title = `${title} | ${default_title}`;
 
-        alternateLinkTagChange();
-        canonicalLinkTagChange();
-
         return result;
     };
 

diff --git a/packages/core/src/index.html b/packages/core/src/index.html
@@ -173,7 +173,7 @@
         <link rel="preconnect" href="https://www.googletagmanager.com" />
         <link rel="preconnect" href="https://browser-update.org" />
 
-        <link rel="alternate" hreflang="en-EN" href="https://app.deriv.com" />
+        <link rel="alternate" hreflang="en" href="https://app.deriv.com" />
         <link rel="canonical" href="https://app.deriv.com" />
 
         <title>Deriv App | Deriv</title>

diff --git a/packages/shared/src/utils/dom/index.ts b/packages/shared/src/utils/dom/index.ts
@@ -1,2 +1 @@
 export * from './position';
-export * from './seo_optimization';
diff --git a/packages/shared/src/utils/dom/seo_optimization.ts b/packages/shared/src/utils/dom/seo_optimization.ts
diff --git a/packages/trader/src/App/Components/Routes/route-with-sub-routes.tsx b/packages/trader/src/App/Components/Routes/route-with-sub-routes.tsx
@@ -1,14 +1,6 @@
 import React from 'react';
 import { Redirect, Route, RouteComponentProps } from 'react-router-dom';
-import {
-    alternateLinkTagChange,
-    canonicalLinkTagChange,
-    redirectToLogin,
-    removeBranchName,
-    routes,
-    isEmptyObject,
-    default_title,
-} from '@deriv/shared';
+import { redirectToLogin, removeBranchName, routes, isEmptyObject, default_title } from '@deriv/shared';
 import { getLanguage } from '@deriv/translations';
 import Page404 from 'Modules/Page404';
 import { TBinaryRoutesProps, TRouteConfig } from 'Types';
@@ -57,9 +49,6 @@ const RouteWithSubRoutes = (route: TRouteWithSubRoutesProps) => {
         const title = route.getTitle?.() ?? '';
         document.title = `${title} | ${default_title}`;
 
-        alternateLinkTagChange();
-        canonicalLinkTagChange();
-
         return result;
     };
Original file line number	Diff line number	Diff line change
		@@ -1,2 +1 @@
		export * from './position';
		export * from './seo_optimization';