-
Notifications
You must be signed in to change notification settings - Fork 893
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
Add migration flow for old grants to new promotions #4101
Conversation
vendor/bat-native-ledger/include/bat/ledger/public/interfaces/ledger.mojom
Outdated
Show resolved
Hide resolved
vendor/bat-native-ledger/src/bat/ledger/internal/common/bind_util.cc
Outdated
Show resolved
Hide resolved
vendor/bat-native-ledger/src/bat/ledger/internal/common/bind_util.cc
Outdated
Show resolved
Hide resolved
d5f03c0
to
3079eff
Compare
3079eff
to
49658b8
Compare
49658b8
to
229c4c3
Compare
CI failed on |
} | ||
if (item->legacy_claimed) { | ||
item->status = ledger::PromotionStatus::CLAIMED; | ||
ClaimTokens(item->Clone(), [](const ledger::Result _){}); |
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.
there is a _
symbol after Result
.
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.
that is needed
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.
Per:
https://google.github.io/styleguide/cppguide.html#Function_Declarations_and_Definitions
A parameter name may be omitted only if the parameter is not used in the function's definition.
The _
is considered as a special symbol. We usually use it as trailing part in class/struct members names.
229c4c3
to
0209aa0
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.
Approving.
Please address (https://github.com/brave/brave-core/pull/4101/files#r353304718) in next PR.
Resolves brave/brave-browser#6937
Submitter Checklist:
npm run lint
)git rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
Reviewer Checklist:
After-merge Checklist:
changes has landed on.