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
Adds different text to grant finish screen #2670
Conversation
"description": "Cosmetic text upon captcha success for ads" | ||
}, | ||
"grantFinishTextAds": { | ||
"message": "Your rewards grant from Brave Ads is on its 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.
needs a full-stop to be consistent with UGP
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.
done
grant.expiryTime = 0 | ||
grant.finishText = getMessage('grantFinishTextAds') |
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.
nit: for consistency swap text/title lines around so they match above
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.
done
@@ -135,14 +135,43 @@ class Grant extends React.Component<Props, State> { | |||
) | |||
} | |||
|
|||
grantFinish = (type: string, tokens: string, date: string) => { | |||
let title = getLocale('grantFinishTitleUGP') |
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.
nit: could we not move to these to an else
statement
return grant | ||
} | ||
|
||
grant.finishTitle = getMessage('grantFinishTitleUGP') |
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.
nit: could we not move to these to an else
statement
@@ -524,6 +522,13 @@ | |||
<message name="IDS_BRAVE_UI_SHOW_ALL" desc="">Show All</message> | |||
<message name="IDS_BRAVE_UI_SUPPORTED_SITES" desc="">Supported Sites</message> | |||
|
|||
<message name="IDS_BRAVE_UI_GRANT_FINISH_TOKEN_ADS" desc="Token text on final grant screen for UGP">Your Brave Ads Token Grant</message> | |||
<message name="IDS_BRAVE_UI_GRANT_FINISH_TEXT_ADS" desc="Text on final grant screen for UGP">Your rewards grant from Brave Ads is on its way</message> |
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.
needs a full-stop to be consistent with UGP
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.
done
e046a5a
to
b55c349
Compare
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.
LGTM
put ads header in BRAVE_ADS_ENABLED guard
put ads header in BRAVE_ADS_ENABLED guard
Resolves brave/brave-browser#4814
UI PR: brave/brave-ui#499
Submitter Checklist:
npm test brave_unit_tests && npm test brave_browser_tests && npm run test-security
) onnpm run lint
)git rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
Test plan 1:
Test plan 1:
Reviewer Checklist:
After-merge Checklist:
changes has landed on.