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

Rewards ads android #477

Closed
wants to merge 17 commits into from
Closed

Rewards ads android #477

wants to merge 17 commits into from

Conversation

petemill
Copy link
Member

@petemill petemill commented May 25, 2019

Rewards feature supported for ads on android (settings page).

Required by android branch at brave/brave-core#2508

Test plan

  • Build is ok
Link / storybook path to visual changes

Integration

  • Does this contain changes to src/components or src/

    • Will you publish to npm immediately after this PR, or wait until sometime in the future?
    • Incompatible API change to something existing (major version increase)
    • Adding new backwards-compatible functionality? (minor version increase)
    • Fixing a bug backwards-compatibly? (patch version increase)
  • Does this contain changes to src/features for brave-core?

    • Are there non backwards-compatible changes required for brave-core? Do not merge until brave-core PR is approvable. Link to brave-core PR: Rewards: string for Ads not supported on device brave-core#2509
    • Will you create brave-core PR to update to this commit after it is merged?
    • Wants uplift to brave-core feature branch?
      • When uplift-approved, merge to brave-core-0.VV.x feature branch
      • Create additional brave-core PRs for each feature branch to update commit

This makes more sense when considering there are different messages for different reasons of ads not being available to the user.

There are corresponding PRs on consuming products to use the renamed IDs
@petemill petemill marked this pull request as ready for review May 28, 2019 05:18
@petemill petemill self-assigned this May 28, 2019
@petemill petemill requested review from ryanml and NejcZdovc May 28, 2019 05:28
@@ -40,6 +12,7 @@ import {
import { List } from '../../'
import { getLocale } from '../../../../helpers'
import Toggle from '../../../../components/formControls/toggle/index'
import * as Styled from './style'
Copy link
Contributor

Choose a reason for hiding this comment

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

🎉

@petemill
Copy link
Member Author

Updated (deployment link included) to add GrantComplete customization

ryanml and others added 4 commits July 16, 2019 21:56
@emerick
Copy link
Collaborator

emerick commented Nov 6, 2019

@petemill Just FYI, I pulled these changes into brave/brave-core#3886.

@NejcZdovc
Copy link
Contributor

closing as this was resolved

@NejcZdovc NejcZdovc closed this Mar 4, 2020
@NejcZdovc NejcZdovc deleted the rewards-ads-android branch March 4, 2020 10:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants