Skip to content

Comments

UHF-12232 Linkedevents performance data to sentry#1460

Merged
rpnykanen merged 6 commits intomainfrom
UHF-12232
Dec 19, 2025
Merged

UHF-12232 Linkedevents performance data to sentry#1460
rpnykanen merged 6 commits intomainfrom
UHF-12232

Conversation

@rpnykanen
Copy link
Contributor

@rpnykanen rpnykanen commented Dec 16, 2025

UHF-12232

What was done

Send linkedevents-performance data to sentry after request.

Etusivu helsinki lähelläsi is one place where you can test this

How to install

  • Make sure your instance is up and running on latest dev-branch
    • git checkout dev && git pull origin dev
    • make fresh
  • Update the HDBT theme
    • composer require drupal/hdbt:dev-UHF-12232
  • Run code updates
    • make drush-locale-update drush-cr
  • Go to admin/config/system/csp and disable the csp, otherwise you cannot send requests to sentry.

How to test

  • Setup local sentry by:
    • setting up local javascript sentry
    • OR
    • add test-helfi-etusivu-react -sentry dns and environment: 'local' to Sentry.ts initSentry -helper function
    • set tracesSampleRate to 1.0 to send all data to sentry
    • and rebuild JS
  • Open network tab
  • Go to the linkedevent search, for example helsinki lähelläsi / tapahtumat -page
  • Check that the sentry request is sent after linkedevents-request
  • You should see something here after a while

@github-actions
Copy link

⚠️ Visual regression found! Please check if this change is wanted or accidental. You can check the output here: https://city-of-helsinki.github.io/drupal-hdbt-visual-regression/pull/1460/html_report/

@rpnykanen rpnykanen merged commit 1838002 into main Dec 19, 2025
6 checks passed
@rpnykanen rpnykanen deleted the UHF-12232 branch December 19, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants