-
Notifications
You must be signed in to change notification settings - Fork 862
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
Automatically limit unwanted Brave Ads delivery #5031
Conversation
@@ -11,6 +11,7 @@ @interface BATAdNotification () | |||
@property (nonatomic, copy) NSString *parentUuid; | |||
@property (nonatomic, copy) NSString *creativeInstanceID; | |||
@property (nonatomic, copy) NSString *creativeSetID; | |||
@property (nonatomic, copy) NSString *campaignID; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see anything in particular (aside from the change to the ads_service_impl at the top), but is this new property supposed to be used in the client in a specific way?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also missing the read-only property definition for this in BATAdsNotification.h
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This new property is not used in the client as of yet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
iOS change LGTM 🙂
11cb6ad
to
67d7025
Compare
24a229b
to
4b2ee90
Compare
20d5af8
to
e7b9e43
Compare
bad3dda
to
801eec3
Compare
CI failed on macOS, restarting macOS |
ba42680
to
8b08be2
Compare
3f7c055
to
16b50c3
Compare
CI failed on Windows for known test, restarting macOS. macOS failing due to known issue at brave/brave-browser#6515 |
Resolves brave/brave-browser#6253
Submitter Checklist:
npm run lint
)git rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
See brave/brave-browser#6253
Reviewer Checklist:
After-merge Checklist:
changes has landed on.