Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Implement out of band updates for Brave Ads #2712

Conversation

tmancey
Copy link
Collaborator

@tmancey tmancey commented Jul 9, 2020

Summary of Changes

This pull request fixes #2707

Submitter Checklist:

  • Unit Tests are updated to cover new or changed functionality
  • User-facing strings use NSLocalizableString()

Test Plan:

Screenshots:

Reviewer Checklist:

  • Issues include necessary QA labels:
    • QA/(Yes|No)
    • release-notes/(include|exclude)
    • bug / enhancement
  • Necessary security reviews have taken place.
  • Adequate unit test coverage exists to prevent regressions.
  • Adequate test plan exists for QA to validate (if applicable).
  • Issue is assigned to a milestone (should happen at merge time).

@tmancey tmancey self-assigned this Jul 9, 2020
brave-core: master (a0bb7d59ed3d0695df46f3e25ec9e53cf8881a97)
latest tag: v1.12.77
brave-browser: master (581b9cde2d15d5cffc8aa9eea1472f9171e3c707)
brave-core: issues/8396 (96586ef16c2bcdabb2d62b019f763c8ecdf1bb1c)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This will have to be built from master before it can be merged

@kylehickinson kylehickinson added the blocked If a ticket is blocked for some reason, if not using a sub-block label, please provide info in issue label Jul 9, 2020
@kylehickinson
Copy link
Collaborator

Have to wait until brave/brave-core#5209 is merged

@kylehickinson kylehickinson force-pushed the brave-core-1.13.x branch 2 times, most recently from db9ea62 to 4eb7b2c Compare July 27, 2020 20:42
@tmancey tmancey force-pushed the brave-core-1.13.x-ios-out-of-band-user-model-updates branch from d6a894f to 43fade9 Compare July 28, 2020 12:07
@kylehickinson
Copy link
Collaborator

Changes only exist in BraveRewards.framework so closing as this will be closed

@kylehickinson kylehickinson deleted the brave-core-1.13.x-ios-out-of-band-user-model-updates branch February 17, 2021 21:31
@kylehickinson kylehickinson restored the brave-core-1.13.x-ios-out-of-band-user-model-updates branch February 17, 2021 21:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
blocked If a ticket is blocked for some reason, if not using a sub-block label, please provide info in issue Epic: Ads
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants