-
Notifications
You must be signed in to change notification settings - Fork 901
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 delays for suggestion api (uplift to 1.3) #4448
Conversation
Adds delays for suggestion api
@tmancey can you help me check this out? I think we're good to go here... it builds, unit tests all pass. I only pulled in the minimum required Once we verify it, we can assign to @brave/uplift-approvers for consideration 😄 |
@bsclifton LGTM++ |
Might still be some additional work. Build worked fine, unit tests all pass. But looks like browser tests have a failure:
|
module ledger.mojom; | ||
|
||
enum ContributionStep { | ||
STEP_AC_TABLE_EMPTY = -4, |
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.
Can I ask why start steps at -4
?
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.
40f63d2
to
9f46417
Compare
OK problem resolved- force pushed change after verifying all tests pass locally! 🎉 |
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++
Looks like |
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.
Uplift into 1.3.x
approved after deliberating with @brave/uplift-approvers. macOS
failed due to test-install
which is a known problem as per #4448 (comment). PR was also checked on Nightly by @NejcZdovc and @LaurenWags as per #4372 (comment).
Please make sure that all the correct labels are being used and the associated issue is moved into the correct milestone.
A more concise manual merge resolution of #4441
Uplift of #4372
Fixes brave/brave-browser#7709
Fixes brave/brave-browser#6940
Approved, please ensure that before merging:
After you merge: