refactor: pass CustomUrl to matomo tag manager pageview events#2887
Conversation
This patch makes the canonical url `<link />` element point to the taxonomy canonical url with basename if available. Also `Pageview` events to matomo tag manager will get the `CustomUrl` variable set to the taxonomy `defaultUrl` so that matomo can use this variable to have shared tracking across language versions. This means the canonical url will now include the language (unless it is `nb`, since that is the same as no language part).
| export const getCanonicalUrl = (pathname: string) => { | ||
| if (!pathname.includes("article-iframe")) { | ||
| return `${config.ndlaFrontendDomain}${pathname}`; | ||
| const parts = pathname.split("/"); |
There was a problem hiding this comment.
Mens du har dette friskt i minnet: Kunne du skrevet et par kommentarer om hvorfor getCanonicalUrl og getAlternateUrl må behandles nn, en og nb forskjellig? Fort gjort å glemme når en ikke holder aktivt på med det.
|
Var ikkje litt av poenget med canonical at det skulle være en adresse som var felles for alle språkvarianter av en artikkel? No vil jo samme artikkel ha forskjellig canonical for forskjellige språkvarianter. |
This custom dimension will allow the backend to lookup urls by subjectId rather than generating the slug.
Det var poenget før jeg hadde full forståelse, men etter å ha gjort litt research så vil det ødelegge litt for SEO, så denne fikser egentlig SEO litt i tillegg. Så canonical vil nå være uten språkkode dersom språket er Men den |
|
Her skal me gjere fleire ting: |
an attempt to validate that we wont get a `undefined` result from unloaded props in the `PageTitle` component.
4 participants
Depends on NDLANO/graphql-api#730
Denne gjør at canonical url
<link />elementet peker påurlfra taksonomi. Med språk fra ndla-frontend (med mindre det ernb).Denne gjør også at
Pageviewevents på matomo tag manager vil fåCustomUrlvariabelen sendt med. Denne skal være satt til defaultUrl fra taksonomi (altså alltid bokmål versjonen).Dette gjør at vi kan tracke språkversjonene likt i matomo.