Skip to content
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

[$4000] Android - Chat - Empty chat (black screen) is displayed instead of messages #17870

Closed
1 of 6 tasks
kbecciv opened this issue Apr 23, 2023 · 151 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Internal Requires API changes or must be handled by Expensify staff

Comments

@kbecciv
Copy link

kbecciv commented Apr 23, 2023

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Action Performed:

  1. Go https://staging.new.expensify.com/
    , login, copy URL of any chat, save it in Email (or slack, or Telegram) for later to tap
  2. Open Android NewDot app (could be logged in or logged out)
  3. Tap deep link from step 1

Expected Result:

Chat should be open with previous history.

Actual Result:

Empty chat (like black screen) is displayed instead of messages

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android / native
  • Android / Chrome
  • iOS / native
  • iOS / Safari
  • MacOS / Chrome / Safari
  • MacOS / Desktop

Version Number: 1.3.4.0

Reproducible in staging?: Yes

Reproducible in production?: Yes

If this was caught during regression testing, add the test name, ID and link from TestRail:

Email or phone of affected tester (no customers):

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Bug6028991_video_60.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~018e845ce5e7422dc1
  • Upwork Job ID: 1654063892620099584
  • Last Price Increase: 2023-05-19
@kbecciv kbecciv added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Apr 23, 2023
@MelvinBot
Copy link

Triggered auto assignment to @maddylewis (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@MelvinBot

This comment was marked as duplicate.

@melvin-bot melvin-bot bot added the Overdue label Apr 26, 2023
@maddylewis
Copy link
Contributor

oopsie! let me try repo'ing this one now!

@melvin-bot melvin-bot bot removed the Overdue label Apr 26, 2023
@maddylewis maddylewis removed their assignment Apr 26, 2023
@maddylewis maddylewis added Bug Something is broken. Auto assigns a BugZero manager. and removed Bug Something is broken. Auto assigns a BugZero manager. labels Apr 26, 2023
@MelvinBot
Copy link

Triggered auto assignment to @dylanexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@MelvinBot
Copy link

MelvinBot commented Apr 26, 2023

Bug0 Triage Checklist (Main S/O)

  • This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
  • This bug is not a duplicate report (check E/App issues and #expensify-bugs)
    • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
  • This bug is reproducible using the reproduction steps in the OP. S/O
    • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
    • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
  • This issue is filled out as thoroughly and clearly as possible
    • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
  • I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@maddylewis
Copy link
Contributor

im still waiting to get access to browserstack so that i can test on an android device - https://expensify.slack.com/archives/C01SKUP7QR0/p1682442907291169?thread_ts=1682442046.213409&cid=C01SKUP7QR0

i am re-adding the bug label to reassign this. @dylanexpensify if you have a recently assigned bug issue, please assign it to me! thank you!

@dylanexpensify
Copy link
Contributor

Woot! gotchu!

@dylanexpensify
Copy link
Contributor

Hmmm can't repro. @kbecciv can you confirm if this is still occurring?

@melvin-bot melvin-bot bot added the Overdue label May 1, 2023
@dylanexpensify
Copy link
Contributor

bump @kbecciv

@melvin-bot melvin-bot bot removed the Overdue label May 2, 2023
@kbecciv
Copy link
Author

kbecciv commented May 2, 2023

Hello @dylanexpensify! Checking, will update you shortly.

@kbecciv
Copy link
Author

kbecciv commented May 2, 2023

@dylanexpensify Issue is reproductible with build 1.3.8.8

video_2023-05-02_12-35-02.1.mp4

@dylanexpensify
Copy link
Contributor

Thanks @kbecciv! Didn't get to this today due to focusing on ECUK and CPM tasks! Will review tomorrow!

@dylanexpensify
Copy link
Contributor

Nice, got it! TY

@dylanexpensify dylanexpensify added the External Added to denote the issue can be worked on by a contributor label May 4, 2023
@melvin-bot melvin-bot bot changed the title Android - Chat - Empty chat (black screen) is displayed instead of messages [$1000] Android - Chat - Empty chat (black screen) is displayed instead of messages May 4, 2023
@MelvinBot
Copy link

Job added to Upwork: https://www.upwork.com/jobs/~018e845ce5e7422dc1

@MelvinBot
Copy link

Current assignee @dylanexpensify is eligible for the External assigner, not assigning anyone new.

@MelvinBot
Copy link

Triggered auto assignment to Contributor-plus team member for initial proposal review - @0xmiroslav (External)

@melvin-bot melvin-bot bot removed the Overdue label Aug 28, 2023
@dylanexpensify
Copy link
Contributor

@hannojg to review responses 🙇‍♂️

@dylanexpensify
Copy link
Contributor

@0xmiroslav any chance of reproing? Adding new engineer

@melvin-bot
Copy link

melvin-bot bot commented Aug 31, 2023

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

@hannojg
Copy link
Contributor

hannojg commented Sep 1, 2023

I think it should be a new issue for "showing a blank before showing the skeleton loading" (which we are currently already working on by fixing the performance for opening a report).

And the issue that a chat never loads isn't reproducible anymore since the onyx changes we made, so I think this issue is done.

@0xmiros
Copy link
Contributor

0xmiros commented Sep 1, 2023

since the onyx changes we made

not only this. There was another root cause in android related to animation

@melvin-bot melvin-bot bot added the Overdue label Sep 4, 2023
@dylanexpensify
Copy link
Contributor

Reviewing today while in the air! Currently taxi'd, will get to once I can hop on wifi!

@melvin-bot melvin-bot bot removed the Overdue label Sep 4, 2023
@dylanexpensify
Copy link
Contributor

Perfect, sounds like we're good to close out? @hannojg @0xmiroslav

@0xmiros
Copy link
Contributor

0xmiros commented Sep 5, 2023

Perfect, sounds like we're good to close out? @hannojg @0xmiroslav

Let's confirm from @quinthar if it's not reproducible anymore on android recently.

@melvin-bot
Copy link

melvin-bot bot commented Sep 11, 2023

@luacmartins, @Piotrfj, @dylanexpensify, @0xmiroslav Huh... This is 4 days overdue. Who can take care of this?

@melvin-bot melvin-bot bot added the Overdue label Sep 11, 2023
@0xmiros
Copy link
Contributor

0xmiros commented Sep 11, 2023

Not overdue

@melvin-bot melvin-bot bot removed the Overdue label Sep 11, 2023
@luacmartins
Copy link
Contributor

@quinthar can you confirm if this bug is still reproducible?

@melvin-bot melvin-bot bot added the Overdue label Sep 13, 2023
@dylanexpensify
Copy link
Contributor

Bump @quinthar

@mallenexpensify
Copy link
Contributor

I'm on the #expensify-open-source weekly update, assigned to @quinthar for 👀 to see if he's able to reproduce.

@melvin-bot melvin-bot bot added the Overdue label Sep 18, 2023
@luacmartins
Copy link
Contributor

Just waiting on confirmation that this is still happening

@melvin-bot melvin-bot bot removed the Overdue label Sep 18, 2023
@quinthar
Copy link
Contributor

Oh hey I'm sorry, I didn't see these bumps as it wasn't assigned to me. I haven't seen this situation in a long time, so I think we're good!

@luacmartins
Copy link
Contributor

Awesome! We're good to close then. Thanks everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Internal Requires API changes or must be handled by Expensify staff
Projects
None yet
Development

No branches or pull requests