This repository has been archived by the owner on Dec 11, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 972
stats ref codes should not be sent at startup #14615
Labels
Milestone
Comments
10 tasks
alexwykoff
added
release/not-blocking
priority/P5
Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative).
priority/P3
Major loss of function.
and removed
priority/P5
Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative).
labels
Jul 3, 2018
ryanml
added a commit
to ryanml/browser-laptop
that referenced
this issue
Jul 12, 2018
ryanml
added a commit
to ryanml/browser-laptop
that referenced
this issue
Jul 12, 2018
ryanml
added a commit
to ryanml/browser-laptop
that referenced
this issue
Jul 15, 2018
ryanml
added a commit
to ryanml/browser-laptop
that referenced
this issue
Jul 15, 2018
alexwykoff
modified the milestones:
0.23.x Release 4,
0.23.x w/ Chromium 68 (BETA Channel)
Jul 17, 2018
ryanml
added a commit
to ryanml/browser-laptop
that referenced
this issue
Jul 19, 2018
ryanml
added a commit
to ryanml/browser-laptop
that referenced
this issue
Jul 19, 2018
@ryanml any verification steps required for this? If not please mark it as |
@srirambv test plan is specified in PR, adding it to this issue as well |
This was referenced Jul 27, 2018
Verified with macOS 10.13 (@kjozwiak ) and 10.12.6 (@LaurenWags) using
Verified on Ubuntu 17.10 x64
|
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
Test Plan:
promoCode
in the profile folder with a test promo code, such askam253
LEDGER_VERBOSE=true and LEDGER_ENVIRONMENT=staging
promoCode
file is deleted when using real promo codeDescription
ledger.js has code for the refcodes to be sent up right away on startup.
We should be avoiding File IO and network requests at startup time in favour of waiting after some initial delay and/or by lazy loading.
In this case, this should instead happen when normal checks happen, see
runtimeUpdateCheckDelay
Steps to Reproduce
2.Start the browser
Actual result:
Network requests happens at startup
Expected result:
Network request shouldn't happen directly at startup.
Reproduces how often:
Always
Brave Version
0.23.19
about:brave info:
Brave: 0.23.19
V8: 6.7.288.46
rev: 178c3fb
Muon: 7.1.3
OS Release: 17.5.0
Update Channel: Release
OS Architecture: x64
OS Platform: macOS
Node.js: 7.9.0
Tor: 0.3.3.7 (git-035a35178c92da94)
Brave Sync: v1.4.2
libchromiumcontent: 67.0.3396.87
Reproducible on current live release:
Yes
Additional Information
The text was updated successfully, but these errors were encountered: