Skip to content

notebooks: fix page view metrics

Administrator requested to merge rn/fix-notebooks-page-view-metrics into main

Created by: novoselrok

Fixes #40850 (closed)

This was a really silent bug that happened when we switched from the deprecated logViewEvent method to the logPageView method in the eventLogger. logViewEvent adds a View prefix to the event name but, for some reason, adds a Viewed suffix. This means you can inadvertently break the usage metrics for pings if you're not careful.

Test plan

  • Covered in the usage metrics test

Merge request reports