Skip to content

[$250] Thread is not loading #53268

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

Closed
1 of 8 tasks
m-natarajan opened this issue Nov 28, 2024 · 34 comments
Closed
1 of 8 tasks

[$250] Thread is not loading #53268

m-natarajan opened this issue Nov 28, 2024 · 34 comments
Assignees
Labels
AutoAssignerNewDotQuality Used to assign quality issues to engineers Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor Needs Reproduction Reproducible steps needed

Comments

@m-natarajan
Copy link

m-natarajan commented Nov 28, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number:
Reproducible in staging?: need reproduction
Reproducible in production?: need reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
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
Expensify/Expensify Issue URL:
Issue reported by: @danielrvidal
Slack conversation (hyperlinked to channel name): Expensify-quality

Action Performed:

  1. Click on a thread https://new.expensify.com/r/1634198864969112

Expected Result:

Thread should open

Actual Result:

Not opening the thread and got a spinner
(it worked before I could turn on loggin.)

Workaround:

unknown

Platforms:

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

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence
2024-11-19_11-21-01.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021863768282323086174
  • Upwork Job ID: 1863768282323086174
  • Last Price Increase: 2024-12-03
@m-natarajan m-natarajan added Daily KSv2 Needs Reproduction Reproducible steps needed Bug Something is broken. Auto assigns a BugZero manager. AutoAssignerNewDotQuality Used to assign quality issues to engineers labels Nov 28, 2024
Copy link

melvin-bot bot commented Nov 28, 2024

Triggered auto assignment to @adelekennedy (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

Copy link

melvin-bot bot commented Nov 28, 2024

Triggered auto assignment to @MariaHCD (AutoAssignerNewDotQuality)

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Nov 28, 2024
@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@melvin-bot melvin-bot bot added the Overdue label Nov 30, 2024
Copy link

melvin-bot bot commented Dec 2, 2024

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

@MariaHCD
Copy link
Contributor

MariaHCD commented Dec 2, 2024

I haven't been able to reproduce this bug thus far 🤔

But let's follow the steps in this SO: https://stackoverflowteams.com/c/expensify/questions/16989

And if we aren't able to reproduce this ourselves, maybe @danielrvidal can try again and if it happens, we can fetch the logs to see if this is a backend issue.

@melvin-bot melvin-bot bot removed the Overdue label Dec 2, 2024
@adelekennedy adelekennedy added the External Added to denote the issue can be worked on by a contributor label Dec 3, 2024
@melvin-bot melvin-bot bot changed the title Thread is not loading [$250] Thread is not loading Dec 3, 2024
Copy link

melvin-bot bot commented Dec 3, 2024

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

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Dec 3, 2024
Copy link

melvin-bot bot commented Dec 3, 2024

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

@adelekennedy adelekennedy removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Dec 3, 2024
@adelekennedy
Copy link

@ntdiary can you give reproducing this a whirl?

@ntdiary
Copy link
Contributor

ntdiary commented Dec 3, 2024

@ntdiary can you give reproducing this a whirl?

@adelekennedy, I can't reproduce it. Maybe it's because this is an private chat?

image

@muttmuure
Copy link
Contributor

This was likely a network issue.

@muttmuure
Copy link
Contributor

We can either:

  • Investigate why the network connection failure was not surfaced in the skeleton
  • Close this out

@bhaggat
Copy link

bhaggat commented Dec 4, 2024

I tried to reproduce it, but could not do it, there must be a specific use case that needs to be debugged by checking all the possibilities.

Possibilities:-

  • Something is broken from the backend, it can be failing or sending the wrong structured data
  • State is not updating for specific use cases after getting the response
  • API call is not triggering for some reason again, can be state-related issue or some condition is blocking the call
  • Sometimes Issue can be because of Structure changed in backend with new version, So it is possible that this is old data and was not patched during updating with new

Way to debug:-

  • Check if the API request is triggering or not, if not go in that direction
  • Check if the response is in the correct structure by checking with other chats, if not fix that issue
  • Check if API request is triggering but it is failing check the issue on the backend by adding logs or comparing it with other similar requests

Copy link

melvin-bot bot commented Dec 4, 2024

📣 @bhaggat! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
    Screen Shot 2022-11-16 at 4 42 54 PM
    Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

@bhaggat
Copy link

bhaggat commented Dec 4, 2024

Contributor details
Your Expensify account email: in [email protected]
Upwork Profile Link: https://www.upwork.com/freelancers/~01b7b8ac0c1016849d

Copy link

melvin-bot bot commented Dec 4, 2024

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

Copy link

melvin-bot bot commented Dec 9, 2024

@ntdiary, @MariaHCD, @adelekennedy Eep! 4 days overdue now. Issues have feelings too...

@melvin-bot melvin-bot bot added the Overdue label Dec 9, 2024
@MariaHCD
Copy link
Contributor

MariaHCD commented Dec 9, 2024

Investigate why the network connection failure was not surfaced in the skeleton

For this, is there a way we can simulate a network connection failure on the frontend? @ntdiary 🤔

@ntdiary
Copy link
Contributor

ntdiary commented Dec 9, 2024

For this, is there a way we can simulate a network connection failure on the frontend? @ntdiary 🤔

@MariaHCD, For the Chrome, we can open the Developer Tools, switch to the Network tab, and select a predefined 4G or 3G or Offline profile. Also, we can use the Add... to customize any download speed. :)

network.mp4

@melvin-bot melvin-bot bot removed the Overdue label Dec 9, 2024
@muttmuure muttmuure moved this to CRITICAL in [#whatsnext] #quality Dec 9, 2024
@ntdiary
Copy link
Contributor

ntdiary commented Dec 10, 2024

Investigate why the network connection failure was not surfaced in the skeleton

@muttmuure, are you referring to the "You appear to be offline" message not being displayed?
image

When we open a thread that hasn’t yet been loaded into the local Onyx, app won't show the main composer, so this message wouldn’t be displayed either, like this:
image

related code:

{isCurrentReportLoadedFromOnyx ? (
<ReportFooter

From the dev tools logs in the OP, it seems their network was changed when opening the thread (i.e., ERR_NETWORK_CHANGED), and then the network was interrupted for a while (i.e., ERR_CONNECTION_REFUSED). As a result, the thread couldn’t be opened, and the offline message wasn’t displayed either.

image

I’m not sure if they were using a proxy or firewall, but I think this should be a very edge case. Additionally, from the original video, it seems their network was reconnected later, allowing them to open the thread normally. :)

Copy link

melvin-bot bot commented Dec 12, 2024

@ntdiary @MariaHCD @adelekennedy this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@melvin-bot melvin-bot bot added the Overdue label Dec 12, 2024
@MariaHCD
Copy link
Contributor

Good points, @ntdiary !

For end users:

  • ERR_CONNECTION_REFUSED indicates a problem with the server or the client's network.
  • ERR_NETWORK_CHANGED suggests the end user's connection changed (e.g., switching between networks).

So this does seem like it's an edge case. I'm not super sure there's anything that we can action here 🤔

Thoughts, @adelekennedy @muttmuure?

Copy link

melvin-bot bot commented Dec 13, 2024

@ntdiary, @MariaHCD, @adelekennedy Whoops! This issue is 2 days overdue. Let's get this updated quick!

Copy link

melvin-bot bot commented Dec 17, 2024

@ntdiary, @MariaHCD, @adelekennedy Still overdue 6 days?! Let's take care of this!

@MariaHCD
Copy link
Contributor

@adelekennedy could you review the comments here and here, and let us know what you think?

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Dec 17, 2024
Copy link

melvin-bot bot commented Dec 23, 2024

@ntdiary, @MariaHCD, @adelekennedy Huh... This is 4 days overdue. Who can take care of this?

@ntdiary
Copy link
Contributor

ntdiary commented Dec 24, 2024

@adelekennedy could you review the comments here and here, and let us know what you think?

waiting for more input, then if this issue still can't be reproduced, maybe we can just close it. :)

@melvin-bot melvin-bot bot removed the Overdue label Dec 24, 2024
@adelekennedy
Copy link

Agreed I think we can likely close this one!

Copy link

melvin-bot bot commented Dec 26, 2024

@ntdiary @MariaHCD @adelekennedy this issue is now 4 weeks old, please consider:

  • Finding a contributor to fix the bug
  • Closing the issue if BZ has been unable to add the issue to a VIP or Wave project
  • If you have any questions, don't hesitate to start a discussion in #expensify-open-source

Thanks!

@melvin-bot melvin-bot bot added the Overdue label Dec 27, 2024
Copy link

melvin-bot bot commented Dec 30, 2024

@ntdiary, @MariaHCD, @adelekennedy Huh... This is 4 days overdue. Who can take care of this?

Copy link

melvin-bot bot commented Jan 1, 2025

@ntdiary, @MariaHCD, @adelekennedy Still overdue 6 days?! Let's take care of this!

Copy link

melvin-bot bot commented Jan 3, 2025

@ntdiary, @MariaHCD, @adelekennedy Now this issue is 8 days overdue. Are you sure this should be a Daily? Feel free to change it!

@MariaHCD
Copy link
Contributor

MariaHCD commented Jan 6, 2025

Looks like we agree that this can be closed since there's nothing we can action here. Closing for now!

@MariaHCD MariaHCD closed this as completed Jan 6, 2025
@github-project-automation github-project-automation bot moved this from CRITICAL to Done in [#whatsnext] #quality Jan 6, 2025
@melvin-bot melvin-bot bot removed Overdue labels Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoAssignerNewDotQuality Used to assign quality issues to engineers Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor Needs Reproduction Reproducible steps needed
Projects
Status: Done
Development

No branches or pull requests

7 participants