-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[ads] Decouple AdsService functionality from iOS BraveAds implementation #42161
Closed
Tracked by
#43892
Labels
dev-concern
feature/ads
OS/iOS
Fixes related to iOS browser functionality
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass - iPhone
QA/Yes
release-notes/exclude
Milestone
Comments
Merged
24 tasks
24 tasks
7 tasks
Verification passed on 1.75.169 on iPhone 13 Pro Max (iOS 18.3) Verified test plan from brave/brave-core#26451 clear dataVerified clicked the search ad:
Verified cleared Ads data: Verified conversions are deleted:
Verified clicked another search ad:
Verified converted the ad:
search adserved
view
click
conversion
NTT adserved
view
click
Brave News Inline adserved
view
click
Notification adserved
view
click
|
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Labels
dev-concern
feature/ads
OS/iOS
Fixes related to iOS browser functionality
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass - iPhone
QA/Yes
release-notes/exclude
This is a prerequisite for the Ads-Internals page task #40952, as we now need to access Ads Service from various parts of the Brave iOS codebase.
The text was updated successfully, but these errors were encountered: