Always return boolean for Snowplow events FE dispatcher
What does this MR do and why?
Related to #358901 (closed)
As a minor improvement suggestion for our Snowplow wrapper, we are now returning a boolean when calling Tracking.{event,definition}
method or the base dispatchSnowplowEvent
function. We returned undefined|Boolean
before, undefined
when the library was called without issues, and false
if Snowplow was disabled.
Do note that currently, there is no logic depending on the return value of this function.
Screenshots or screen recordings
No visual changes.
How to set up and validate locally
- Follow the instructions in our docs to enable Snowplow locally if you haven't done so.
- Trigger any Snowplow event.
- Verify the Snowplow testing tool of your choice, and it should be dispatched without issues.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Axel García