App
App copied to clipboard
Members list blinking when user goes from offline to online - reported by @gadhiyamanan
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
- turn off Internet Connection
- Go to settings > Workspace >Manage Members
- turn on Internet Connection
- check members list
Expected Result:
Members list should not blink
Actual Result:
Members list should blinking
Workaround:
unknown
Platform:
Where is this issue occurring?
- iOS
- Android
Version Number: 1.1.99-0 Reproducible in staging?: Y Reproducible in production?: Y Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/190949938-62dc9b37-da1c-450e-90be-a73988b05f41.mp4
Expensify/Expensify Issue URL: Issue reported by: @gadhiyamanan Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1661604591133289
Triggered auto assignment to @Christinadobrzyn (AutoAssignerTriage
), see https://stackoverflow.com/c/expensify/questions/4749 for more details.
oh interesting, I'm not sure if this will be part of the bigger offline project. I can't find any other GHs related to this but going to get a second opinion from eng about this fix happening now or later.
Triggered auto assignment to @stitesExpensify (Engineering
), see https://stackoverflow.com/c/expensify/questions/4319 for more details.
cc @vitHoracek do you know if this is already being worked on in the policy refactor?
@stitesExpensify I havent worked on this one specifically so I am not sure. I think it was @neil-marcellini who worked on policy related refactorings.
However, I have seen this blinking on unstable wifi in multiple components, so this actually might be related to the offline indicator.
Here is the Workspace refactor tracking issue. The only thing that has been changed about the workspace members so far is this issue for GetFullPolicy.
So it looks like we will be doing some work on the members list page then, should we just put this on hold until those are done?
@stitesExpensify Huh... This is 4 days overdue. Who can take care of this?
bump @neil-marcellini any thoughts on :that: ?
Hmm I'm really not sure. I don't think we should hold on this, it seems unrelated to any refactors we are doing. cc @arosiclair and @chiragsalian.
Yeah I'm not sure what could be causing this either. FWIW I can't repro this on Android v1.2.5-2
hmm do you know if this is still happening @mvtglobally ?
@stitesExpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!
Waiting to see if this is still reproducible
Still waiting for confirmation that this is broken
@stitesExpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
I think it does not need to be daily now as we wait to see what are the reproduction steps
@mvtglobally is this still on the list of issues we're testing?
bump @mvtglobally
@stitesExpensify Team checked on recent build and issue is not repro https://user-images.githubusercontent.com/43995119/199122285-b888b733-40df-4986-ab3a-b77967363a88.MOV https://user-images.githubusercontent.com/43995119/199122306-66d77a80-bc5b-408f-b4b5-dc1fbabd1f83.MOV
Great, thanks!