-
-
Notifications
You must be signed in to change notification settings - Fork 773
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
decreaseses the login from from 7s to 2.3s by making the sync_profile tasks happen on a delay #7652
Conversation
Codecov Report
@@ Coverage Diff @@
## stable #7652 +/- ##
==========================================
- Coverage 25.75% 25.74% -0.01%
==========================================
Files 324 324
Lines 32420 32436 +16
Branches 4798 4799 +1
==========================================
+ Hits 8349 8352 +3
- Misses 23783 23791 +8
- Partials 288 293 +5
Continue to review full report at Codecov.
|
actually_sync_profile(handle, user=user, hide_profile=hide_profile) | ||
|
||
|
||
def actually_sync_profile(handle, user=None, hide_profile=True): |
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.
many of the chat related tasks here could be done on a delay as well; a further PR would possibly do that so that upon first login were not blocking on things like chat tasks to create the profile
getting this into master and then into stable for this weeks deploy |
Description
decreaseses the login from from 7s to 2.3s by making the sync tasks happen on a delay
Refers/Fixes
self evident
Testing
tested locally