App icon indicating copy to clipboard operation
App copied to clipboard

Members list blinking when user goes from offline to online - reported by @gadhiyamanan

Open mvtglobally opened this issue 2 years ago • 18 comments

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:

  1. turn off Internet Connection
  2. Go to settings > Workspace >Manage Members
  3. turn on Internet Connection
  4. 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

View all open jobs on GitHub

mvtglobally avatar Sep 19 '22 04:09 mvtglobally

Triggered auto assignment to @Christinadobrzyn (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

melvin-bot[bot] avatar Sep 19 '22 04:09 melvin-bot[bot]

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.

Christinadobrzyn avatar Sep 19 '22 05:09 Christinadobrzyn

Triggered auto assignment to @stitesExpensify (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

melvin-bot[bot] avatar Sep 19 '22 05:09 melvin-bot[bot]

cc @vitHoracek do you know if this is already being worked on in the policy refactor?

stitesExpensify avatar Sep 19 '22 21:09 stitesExpensify

@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.

mountiny avatar Sep 20 '22 08:09 mountiny

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.

neil-marcellini avatar Sep 20 '22 15:09 neil-marcellini

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 avatar Sep 20 '22 16:09 stitesExpensify

@stitesExpensify Huh... This is 4 days overdue. Who can take care of this?

melvin-bot[bot] avatar Sep 26 '22 06:09 melvin-bot[bot]

bump @neil-marcellini any thoughts on :that: ?

stitesExpensify avatar Sep 26 '22 16:09 stitesExpensify

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.

neil-marcellini avatar Sep 27 '22 15:09 neil-marcellini

Yeah I'm not sure what could be causing this either. FWIW I can't repro this on Android v1.2.5-2

arosiclair avatar Sep 27 '22 16:09 arosiclair

hmm do you know if this is still happening @mvtglobally ?

stitesExpensify avatar Sep 29 '22 18:09 stitesExpensify

@stitesExpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!

melvin-bot[bot] avatar Oct 03 '22 06:10 melvin-bot[bot]

Waiting to see if this is still reproducible

stitesExpensify avatar Oct 03 '22 17:10 stitesExpensify

Still waiting for confirmation that this is broken

stitesExpensify avatar Oct 06 '22 14:10 stitesExpensify

@stitesExpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

melvin-bot[bot] avatar Oct 10 '22 06:10 melvin-bot[bot]

I think it does not need to be daily now as we wait to see what are the reproduction steps

mountiny avatar Oct 10 '22 08:10 mountiny

@mvtglobally is this still on the list of issues we're testing?

stitesExpensify avatar Oct 20 '22 21:10 stitesExpensify

bump @mvtglobally

stitesExpensify avatar Oct 31 '22 22:10 stitesExpensify

@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

mvtglobally avatar Oct 31 '22 22:10 mvtglobally

Great, thanks!

stitesExpensify avatar Oct 31 '22 22:10 stitesExpensify