Closed nsingh-branch closed 6 months ago
Attention: Patch coverage is 16.66667%
with 10 lines
in your changes are missing coverage. Please review.
Project coverage is 38.04%. Comparing base (
53e1788
) to head (465a120
).:exclamation: Current head 465a120 differs from pull request most recent head cb0a21e. Consider uploading reports for the commit cb0a21e to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
The server actually needs to know if a Meta Install Referrer was a click-through or view-through, so we need to pass it through in the install request. To do this, we create a new field in install requests called
is_meta_ct
. It will only be included in installs where a Meta Install Referrer is being attached as well.Testing Instructions
Check that the field is being included in the request only when necessary and accurately. Also ensure no other logic is being broken by this change.
Risk Assessment [
LOW
]Reviewer Checklist (To be checked off by the reviewer only)
cc @BranchMetrics/saas-sdk-devs for visibility.