-
Notifications
You must be signed in to change notification settings - Fork 1
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
MIM-1887 gtag and GA_TRACKING_ID removal #2769
Conversation
Bumps [google-github-actions/auth](https://github.com/google-github-actions/auth) from 2.1.2 to 2.1.3. - [Release notes](https://github.com/google-github-actions/auth/releases) - [Changelog](https://github.com/google-github-actions/auth/blob/main/CHANGELOG.md) - [Commits](google-github-actions/auth@v2.1.2...v2.1.3) --- updated-dependencies: - dependency-name: google-github-actions/auth dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer
* added macro config for keyFigure * added styling to fix gap to text * fixed formatting
* Updated comment for GA4 gtag config * Fjernet UA script
src/main/resources/site/parts/searchResult/searchResultView.tsx
Outdated
Show resolved
Hide resolved
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.
Se på mine kommentarer
ba60330
to
8b44cfa
Compare
@@ -66,7 +66,6 @@ const previewOverride: object = { | |||
contentList: 'relatedFactPage', | |||
} | |||
|
|||
export const GA_TRACKING_ID: string | null = app.config?.GA_TRACKING_ID || null |
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.
Siden vi henter denne fra config betyr det at vi må rydde bort GA_TRACKING_ID i mimir-config-repo også. Må vel gjøres etter at vi har deployet til alle miljøene for å gjøre det enkelt :)
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.
Ser bra ut! Vent med å merge til vi ser at testene for test-miljøet går gjennom for denne PR-en. Og så må vi huske på å lage en sak på å rydde bort ID-en i mimir-config.
Quality Gate passedIssues Measures |
* Add PageMap attributes, feature toggle * [MIM-1950] MIM-1950:Fikset bug med tidssone for kommende publiseringer (#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer * Change env name in github action to make jira integration work properly (#2764) * [MIM-1915] Added macro config for keyFigure (#2759) * added macro config for keyFigure * added styling to fix gap to text * fixed formatting * Add conditional fields, if data is present. * [MIM-1887] Mim 1887 removal of universal analytics (#2763) * Updated comment for GA4 gtag config * Fjernet UA script * Added description and title after code review * Keywords should be an array, I think. * Remove title, it looks wrong sometimes and shouldnt be fetched like this. * [MIM-1887] UA script for TEST (#2772) UA script for TEST * MIM-1887 gtag and GA_TRACKING_ID removal (#2769) * highchart.es6 gtag removal * AttachmentTablesFigures gtag removal * Highchart.jsx gtag removal * links.tsx gtag removal * Table.tsx gtag removal * nameSearch.jsx gtag removal * searchResultView.tsx gtag removal * error.html gtag removal * ReactGA.tsx removal of file * react-ga dependency removal * GA_TRACKING_ID removal from all files * Bump google-github-actions/auth from 2.1.2 to 2.1.3 (#2757) Bumps [google-github-actions/auth](https://github.com/google-github-actions/auth) from 2.1.2 to 2.1.3. - [Release notes](https://github.com/google-github-actions/auth/releases) - [Changelog](https://github.com/google-github-actions/auth/blob/main/CHANGELOG.md) - [Commits](google-github-actions/auth@v2.1.2...v2.1.3) --- updated-dependencies: - dependency-name: google-github-actions/auth dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Update version number to v2.8.2 * [MIM-1950] MIM-1950:Fikset bug med tidssone for kommende publiseringer (#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer * Change env name in github action to make jira integration work properly (#2764) * [MIM-1915] Added macro config for keyFigure (#2759) * added macro config for keyFigure * added styling to fix gap to text * fixed formatting * [MIM-1887] Mim 1887 removal of universal analytics (#2763) * Updated comment for GA4 gtag config * Fjernet UA script * Changes to Commits * Added type for renderShowButton onKeyDown event * default.html - UA SCRIPT final removal --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: GitHub Actions Bot <> Co-authored-by: Carina <47381367+ssb-cgn@users.noreply.github.com> Co-authored-by: Anne-Siri <142512638+annesiri@users.noreply.github.com> * [MIM-1915] Add divider to key figure macro (#2775) * added divider in keyFigure macro * moved divider logic to keyFigure part --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Glenruben Larsen <gle@ssb.no> Co-authored-by: Carina <47381367+ssb-cgn@users.noreply.github.com> Co-authored-by: Anne-Siri <142512638+annesiri@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
PROD Løft (#2776) * Add PageMap attributes, feature toggle * [MIM-1950] MIM-1950:Fikset bug med tidssone for kommende publiseringer (#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer * Change env name in github action to make jira integration work properly (#2764) * [MIM-1915] Added macro config for keyFigure (#2759) * added macro config for keyFigure * added styling to fix gap to text * fixed formatting * Add conditional fields, if data is present. * [MIM-1887] Mim 1887 removal of universal analytics (#2763) * Updated comment for GA4 gtag config * Fjernet UA script * Added description and title after code review * Keywords should be an array, I think. * Remove title, it looks wrong sometimes and shouldnt be fetched like this. * [MIM-1887] UA script for TEST (#2772) UA script for TEST * MIM-1887 gtag and GA_TRACKING_ID removal (#2769) * highchart.es6 gtag removal * AttachmentTablesFigures gtag removal * Highchart.jsx gtag removal * links.tsx gtag removal * Table.tsx gtag removal * nameSearch.jsx gtag removal * searchResultView.tsx gtag removal * error.html gtag removal * ReactGA.tsx removal of file * react-ga dependency removal * GA_TRACKING_ID removal from all files * Bump google-github-actions/auth from 2.1.2 to 2.1.3 (#2757) Bumps [google-github-actions/auth](https://github.com/google-github-actions/auth) from 2.1.2 to 2.1.3. - [Release notes](https://github.com/google-github-actions/auth/releases) - [Changelog](https://github.com/google-github-actions/auth/blob/main/CHANGELOG.md) - [Commits](google-github-actions/auth@v2.1.2...v2.1.3) --- updated-dependencies: - dependency-name: google-github-actions/auth dependency-type: direct:production update-type: version-update:semver-patch ... * Update version number to v2.8.2 * [MIM-1950] MIM-1950:Fikset bug med tidssone for kommende publiseringer (#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer * Change env name in github action to make jira integration work properly (#2764) * [MIM-1915] Added macro config for keyFigure (#2759) * added macro config for keyFigure * added styling to fix gap to text * fixed formatting * [MIM-1887] Mim 1887 removal of universal analytics (#2763) * Updated comment for GA4 gtag config * Fjernet UA script * Changes to Commits * Added type for renderShowButton onKeyDown event * default.html - UA SCRIPT final removal --------- Co-authored-by: GitHub Actions Bot <> * [MIM-1915] Add divider to key figure macro (#2775) * added divider in keyFigure macro * moved divider logic to keyFigure part --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Carl-OW <142233642+Carl-OW@users.noreply.github.com> Co-authored-by: Carina <47381367+ssb-cgn@users.noreply.github.com> Co-authored-by: Anne-Siri <142512638+annesiri@users.noreply.github.com> Co-authored-by: Glenruben <gle@ssb.no> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* highchart.es6 gtag removal * AttachmentTablesFigures gtag removal * Highchart.jsx gtag removal * links.tsx gtag removal * Table.tsx gtag removal * nameSearch.jsx gtag removal * searchResultView.tsx gtag removal * error.html gtag removal * ReactGA.tsx removal of file * react-ga dependency removal * GA_TRACKING_ID removal from all files * Bump google-github-actions/auth from 2.1.2 to 2.1.3 (#2757) Bumps [google-github-actions/auth](https://github.com/google-github-actions/auth) from 2.1.2 to 2.1.3. - [Release notes](https://github.com/google-github-actions/auth/releases) - [Changelog](https://github.com/google-github-actions/auth/blob/main/CHANGELOG.md) - [Commits](google-github-actions/auth@v2.1.2...v2.1.3) --- updated-dependencies: - dependency-name: google-github-actions/auth dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Update version number to v2.8.2 * [MIM-1950] MIM-1950:Fikset bug med tidssone for kommende publiseringer (#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer * Change env name in github action to make jira integration work properly (#2764) * [MIM-1915] Added macro config for keyFigure (#2759) * added macro config for keyFigure * added styling to fix gap to text * fixed formatting * [MIM-1887] Mim 1887 removal of universal analytics (#2763) * Updated comment for GA4 gtag config * Fjernet UA script * Changes to Commits * Added type for renderShowButton onKeyDown event * default.html - UA SCRIPT final removal --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: GitHub Actions Bot <> Co-authored-by: Carina <47381367+ssb-cgn@users.noreply.github.com> Co-authored-by: Anne-Siri <142512638+annesiri@users.noreply.github.com>
PROD Løft (#2776) * Add PageMap attributes, feature toggle * [MIM-1950] MIM-1950:Fikset bug med tidssone for kommende publiseringer (#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer * Change env name in github action to make jira integration work properly (#2764) * [MIM-1915] Added macro config for keyFigure (#2759) * added macro config for keyFigure * added styling to fix gap to text * fixed formatting * Add conditional fields, if data is present. * [MIM-1887] Mim 1887 removal of universal analytics (#2763) * Updated comment for GA4 gtag config * Fjernet UA script * Added description and title after code review * Keywords should be an array, I think. * Remove title, it looks wrong sometimes and shouldnt be fetched like this. * [MIM-1887] UA script for TEST (#2772) UA script for TEST * MIM-1887 gtag and GA_TRACKING_ID removal (#2769) * highchart.es6 gtag removal * AttachmentTablesFigures gtag removal * Highchart.jsx gtag removal * links.tsx gtag removal * Table.tsx gtag removal * nameSearch.jsx gtag removal * searchResultView.tsx gtag removal * error.html gtag removal * ReactGA.tsx removal of file * react-ga dependency removal * GA_TRACKING_ID removal from all files * Bump google-github-actions/auth from 2.1.2 to 2.1.3 (#2757) Bumps [google-github-actions/auth](https://github.com/google-github-actions/auth) from 2.1.2 to 2.1.3. - [Release notes](https://github.com/google-github-actions/auth/releases) - [Changelog](https://github.com/google-github-actions/auth/blob/main/CHANGELOG.md) - [Commits](google-github-actions/auth@v2.1.2...v2.1.3) --- updated-dependencies: - dependency-name: google-github-actions/auth dependency-type: direct:production update-type: version-update:semver-patch ... * Update version number to v2.8.2 * [MIM-1950] MIM-1950:Fikset bug med tidssone for kommende publiseringer (#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer * Change env name in github action to make jira integration work properly (#2764) * [MIM-1915] Added macro config for keyFigure (#2759) * added macro config for keyFigure * added styling to fix gap to text * fixed formatting * [MIM-1887] Mim 1887 removal of universal analytics (#2763) * Updated comment for GA4 gtag config * Fjernet UA script * Changes to Commits * Added type for renderShowButton onKeyDown event * default.html - UA SCRIPT final removal --------- Co-authored-by: GitHub Actions Bot <> * [MIM-1915] Add divider to key figure macro (#2775) * added divider in keyFigure macro * moved divider logic to keyFigure part --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Carl-OW <142233642+Carl-OW@users.noreply.github.com> Co-authored-by: Carina <47381367+ssb-cgn@users.noreply.github.com> Co-authored-by: Anne-Siri <142512638+annesiri@users.noreply.github.com> Co-authored-by: Glenruben <gle@ssb.no> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
PROD Løft (#2776) * Add PageMap attributes, feature toggle * [MIM-1950] MIM-1950:Fikset bug med tidssone for kommende publiseringer (#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer * Change env name in github action to make jira integration work properly (#2764) * [MIM-1915] Added macro config for keyFigure (#2759) * added macro config for keyFigure * added styling to fix gap to text * fixed formatting * Add conditional fields, if data is present. * [MIM-1887] Mim 1887 removal of universal analytics (#2763) * Updated comment for GA4 gtag config * Fjernet UA script * Added description and title after code review * Keywords should be an array, I think. * Remove title, it looks wrong sometimes and shouldnt be fetched like this. * [MIM-1887] UA script for TEST (#2772) UA script for TEST * MIM-1887 gtag and GA_TRACKING_ID removal (#2769) * highchart.es6 gtag removal * AttachmentTablesFigures gtag removal * Highchart.jsx gtag removal * links.tsx gtag removal * Table.tsx gtag removal * nameSearch.jsx gtag removal * searchResultView.tsx gtag removal * error.html gtag removal * ReactGA.tsx removal of file * react-ga dependency removal * GA_TRACKING_ID removal from all files * Bump google-github-actions/auth from 2.1.2 to 2.1.3 (#2757) Bumps [google-github-actions/auth](https://github.com/google-github-actions/auth) from 2.1.2 to 2.1.3. - [Release notes](https://github.com/google-github-actions/auth/releases) - [Changelog](https://github.com/google-github-actions/auth/blob/main/CHANGELOG.md) - [Commits](google-github-actions/auth@v2.1.2...v2.1.3) --- updated-dependencies: - dependency-name: google-github-actions/auth dependency-type: direct:production update-type: version-update:semver-patch ... * Update version number to v2.8.2 * [MIM-1950] MIM-1950:Fikset bug med tidssone for kommende publiseringer (#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer * Change env name in github action to make jira integration work properly (#2764) * [MIM-1915] Added macro config for keyFigure (#2759) * added macro config for keyFigure * added styling to fix gap to text * fixed formatting * [MIM-1887] Mim 1887 removal of universal analytics (#2763) * Updated comment for GA4 gtag config * Fjernet UA script * Changes to Commits * Added type for renderShowButton onKeyDown event * default.html - UA SCRIPT final removal --------- Co-authored-by: GitHub Actions Bot <> * [MIM-1915] Add divider to key figure macro (#2775) * added divider in keyFigure macro * moved divider logic to keyFigure part --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Carl-OW <142233642+Carl-OW@users.noreply.github.com> Co-authored-by: Carina <47381367+ssb-cgn@users.noreply.github.com> Co-authored-by: Anne-Siri <142512638+annesiri@users.noreply.github.com> Co-authored-by: Glenruben <gle@ssb.no> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
PROD Løft (#2776) * Add PageMap attributes, feature toggle * [MIM-1950] MIM-1950:Fikset bug med tidssone for kommende publiseringer (#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer * Change env name in github action to make jira integration work properly (#2764) * [MIM-1915] Added macro config for keyFigure (#2759) * added macro config for keyFigure * added styling to fix gap to text * fixed formatting * Add conditional fields, if data is present. * [MIM-1887] Mim 1887 removal of universal analytics (#2763) * Updated comment for GA4 gtag config * Fjernet UA script * Added description and title after code review * Keywords should be an array, I think. * Remove title, it looks wrong sometimes and shouldnt be fetched like this. * [MIM-1887] UA script for TEST (#2772) UA script for TEST * MIM-1887 gtag and GA_TRACKING_ID removal (#2769) * highchart.es6 gtag removal * AttachmentTablesFigures gtag removal * Highchart.jsx gtag removal * links.tsx gtag removal * Table.tsx gtag removal * nameSearch.jsx gtag removal * searchResultView.tsx gtag removal * error.html gtag removal * ReactGA.tsx removal of file * react-ga dependency removal * GA_TRACKING_ID removal from all files * Bump google-github-actions/auth from 2.1.2 to 2.1.3 (#2757) Bumps [google-github-actions/auth](https://github.com/google-github-actions/auth) from 2.1.2 to 2.1.3. - [Release notes](https://github.com/google-github-actions/auth/releases) - [Changelog](https://github.com/google-github-actions/auth/blob/main/CHANGELOG.md) - [Commits](google-github-actions/auth@v2.1.2...v2.1.3) --- updated-dependencies: - dependency-name: google-github-actions/auth dependency-type: direct:production update-type: version-update:semver-patch ... * Update version number to v2.8.2 * [MIM-1950] MIM-1950:Fikset bug med tidssone for kommende publiseringer (#2756) MIM-1950:Fikset bug med tidssone for kommende publiseringer * Change env name in github action to make jira integration work properly (#2764) * [MIM-1915] Added macro config for keyFigure (#2759) * added macro config for keyFigure * added styling to fix gap to text * fixed formatting * [MIM-1887] Mim 1887 removal of universal analytics (#2763) * Updated comment for GA4 gtag config * Fjernet UA script * Changes to Commits * Added type for renderShowButton onKeyDown event * default.html - UA SCRIPT final removal --------- Co-authored-by: GitHub Actions Bot <> * [MIM-1915] Add divider to key figure macro (#2775) * added divider in keyFigure macro * moved divider logic to keyFigure part --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Carl-OW <142233642+Carl-OW@users.noreply.github.com> Co-authored-by: Carina <47381367+ssb-cgn@users.noreply.github.com> Co-authored-by: Anne-Siri <142512638+annesiri@users.noreply.github.com> Co-authored-by: Glenruben <gle@ssb.no> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Link to ticket: MIM-1887