Fix: adding isCanonical lately after removing falsy items #72
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the Change
In #71, we changed the tracking data sent to Sophi collector to send
isCanonical
value. This change introduces another issue: we're sending empty array andfalse
values as well. We only need a boolean value forisCanonical
field, so this PR reverts the change made in #71 and adds theisCanonical
key after filtering the tracking data.Verification Process
Ensure the tracking data doesn't contain empty strings, empty arrays, and
false
(exceptisCanonical
field).Checklist:
Applicable Issues
Changelog Entry