Skip to content

[feat] Deprecate eventLogger.logViewEvent for a new method called eventLogger.logPageView

Warren Gifford requested to merge milan-fix-logviewevent into main

Created by: kopancek

Description

Previously, logViewEvent method was logging events in format View${PageName}, however the new convention is to give the verb as a suffix instead of a prefix, e.g. ${PageName}Viewed. The new logPageView function follows the new convention.

Test plan

These are changes in event tracking, which are almost of impossible to test locally. The change is mostly about deprecating a method and creating a new one to replace it. Only used the new method in part of the code that is behind feature flags. So should be pretty safe. The only change in new method implementation is a new string format for the event name, that follows new conventions.

Merge request reports

Loading