-
Notifications
You must be signed in to change notification settings - Fork 23
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
feat(misc): Profile picture/banner split #1775
Conversation
UI Automated Test Results Summary for MacOS/Windows502 tests 450 ✅ 2h 14m 11s ⏱️ Results for commit 94be621. ♻️ This comment has been updated with latest results. |
UI Automated Tests execution is complete! You can find the test results report here |
If im understanding this statement, then for your own profile, you shouldnt need to concurrently fetch your own identity after updating your profile since the purpose of this is to improve the initial load time by offloading the loading of images into a separate task and update the state with that data if its available. If you are to stick with using this while updating your profile then you might have to wait a bit because it might be that the state is being updated out of order |
What this PR does 📖
Which issue(s) this PR fixes 🔨