Skip to content
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

Include a random delay between sending fee and tips for p2p contributions #5334

Closed
evq opened this issue Jul 23, 2019 · 7 comments · Fixed by brave/brave-core#3388
Closed

Comments

@evq
Copy link
Member

evq commented Jul 23, 2019

Description

Once the publishers API parts of https://github.com/brave-intl/creators-private-issues/issues/623 are rolled out we should add a random delay between sending the fee and tip

@GeetaSarvadnya
Copy link

@NejcZdovc Can you please include the test plan?

@NejcZdovc
Copy link
Contributor

@GeetaSarvadnya added

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Sep 25, 2019

@NejcZdovc Verified the test plan from brave/brave-core#3388. After tipping a verified pub in staging env there's just 3 to 4 secs of delay to reflect the 5% fee in the brave card. The delay is not Noticeable (by the time I navigate from brave://rewards to ledgers sandbox tab 3 secs will over) step 3 in the test plan says to wait until 5% fee card reflect. Is 3 to 4 sec is the expected delay?

5% fee card before tipping a verified
image

5% fee card after tipping a verified pub

image

@NejcZdovc
Copy link
Contributor

delay is random, so yeah could be 4sec as well

@GeetaSarvadnya
Copy link

Verification passed on

Brave 0.69.128 Chromium: 77.0.3865.75 (Official Build) (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS Windows 10 OS Version 1803 (Build 17134.1006)

NejcZdovc added a commit to brave/brave-core that referenced this issue Sep 26, 2019
@kjozwiak
Copy link
Member

kjozwiak commented Sep 27, 2019

@NejcZdovc did this land in 0.69.x? I'm just about to start the approval process for brave/brave-core#3543 which has yet to uplifted so assuming the above verification for window is invalid and needs to be rechecked once brave/brave-core#3543 lands. Clearing QA Pass-Win64.

@NejcZdovc please let me know if I'm wrong or have missed something 👍

@LaurenWags
Copy link
Member

LaurenWags commented Oct 1, 2019

Verified passed with

Brave 0.69.130 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS macOS Version 10.13.6 (Build 17G5019)

Screen Shot 2019-10-01 at 9 25 09 AM

Verification passed on

Brave 0.69.130 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Windows 10 OS Version 1803 (Build 17134.1006)

image

image

image

Verification passed on

Brave 0.69.130 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Ubuntu 18.04 LTS

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants