Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Newform AppsFlyer Integration] Add AppsFlyer SDK, adapter and ATT #2894

Merged
merged 10 commits into from
Mar 27, 2025

Conversation

danielebogo
Copy link
Contributor

📘 Part of: #2850

Fixes #2873
Fixes #2874

This PR implements:
• Adds the new SDK as dynamic SPM
• Create the new adapter
• Creates a new AnonymousIdentifiable protocol to implement the anon UUID

To test

• CI must be 🟢
• Make sure you set true these 2 FF: podcastNewformAppsFlyer and appsFlyerLogging
• Make sure you are logged in

Give Consent

• Run the app
• Confirm you see the ATT prompted
• Give consent
• Navigate a bit in the app
• You should see the AppsFlyer SDK debug log

Don't Give Consent

• Navigate to your System Settings -> Privacy -> Tracking and disable the consent
• Re-run the app (this settings change will kill the debugger)
• Confirm you don't see any SDK logs

Fresh Install

• Delete the app
• Re-run
• Confirm you see again the ATT prompt
• Give consent
• Confirm you see the logs

Checklist

  • I have considered if this change warrants user-facing release notes and have added them to CHANGELOG.md if necessary.
  • I have considered adding unit tests for my changes.
  • I have updated (or requested that someone edit) the spreadsheet to reflect any new or changed analytics.

@danielebogo danielebogo added this to the 7.86 milestone Mar 26, 2025
@danielebogo danielebogo requested a review from a team as a code owner March 26, 2025 17:19
@danielebogo danielebogo changed the title [Newform AppsFlyer Integration] /2873 add sdk and tracker [Newform AppsFlyer Integration] Add AppsFlyer SDK, adapter and ATT Mar 26, 2025
Copy link
Contributor

@SergioEstevao SergioEstevao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check all steps, logs show when consent is given and do not show when disabled.
:shipit:

Copy link
Contributor

@SergioEstevao SergioEstevao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@danielebogo danielebogo merged commit 8fca74a into trunk Mar 27, 2025
4 of 6 checks passed
@danielebogo danielebogo deleted the danieleb/AppsFlyer-2850/2873-add-sdk-and-tracker branch March 27, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement ATT dialog Implement SDK and create new AnalyticsAdapter
2 participants