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

[$1000] Desktop - Chat - $NaN instead of value after red dot error received #18852

Closed
1 of 6 tasks
kbecciv opened this issue May 12, 2023 · 12 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Needs Reproduction Reproducible steps needed

Comments

@kbecciv
Copy link

kbecciv commented May 12, 2023

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


Issue found when executing #18328

Action Performed:

Preconditions:
main user: individual policy with USD currency
secondary user. individual policy with UAH currency

  1. In the main testing device - Disable the internet connection
  2. As the secondary user - Start a conversation with the user from the main testing device, send some messages
  3. In the main testing device - click on the global create + Request money
  4. Follow the prompts send IOU to the secondary user
  5. Verify the IOU Preview and message appear greyed out
  6. Enable the internet connection in the main testing
  7. Verify you see a solid red dot on the chat row of the report that failed to create
  8. Make sure there is no "Unexpected error" below the IOU

Expected Result:

Make sure there is no "Unexpected error" below the IOU
Click/tap the X to dismiss the error message at the top of the report
Make sure that the report goes away as well as from the LHN

Actual Result:

$NaN instead of value after red dot error received

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

Reproducible in staging?: Yes

Reproducible in production?: No

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

Bug6052793_Screen_Recording_2023-05-12_at_04.49.44.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/~01cd243caa0df074df
  • Upwork Job ID: 1657766119867101184
  • Last Price Increase: 2023-05-14
@kbecciv kbecciv added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels May 12, 2023
@melvin-bot
Copy link

melvin-bot bot commented May 12, 2023

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

@melvin-bot
Copy link

melvin-bot bot commented May 12, 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

@mountiny
Copy link
Contributor

Not sure if its necessarily related directly to that PR. I think we can make this external to figure this one out.

@mountiny mountiny added the External Added to denote the issue can be worked on by a contributor label May 14, 2023
@melvin-bot melvin-bot bot changed the title Desktop - Chat - $NaN instead of value after red dot error received [$1000] Desktop - Chat - $NaN instead of value after red dot error received May 14, 2023
@melvin-bot
Copy link

melvin-bot bot commented May 14, 2023

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

@melvin-bot
Copy link

melvin-bot bot commented May 14, 2023

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

@melvin-bot
Copy link

melvin-bot bot commented May 14, 2023

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

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label May 14, 2023
@melvin-bot
Copy link

melvin-bot bot commented May 14, 2023

Triggered auto assignment to @joelbettner (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@tienifr
Copy link
Contributor

tienifr commented May 15, 2023

I think this issue will be fixed after #15297 is merged

@melvin-bot melvin-bot bot added the Overdue label May 15, 2023
@s77rt
Copy link
Contributor

s77rt commented May 15, 2023

@tienifr Can you please elaborate? Do we also have a duplicated request here too?
BTW, are you able to reproduce this bug?

@melvin-bot melvin-bot bot removed the Overdue label May 15, 2023
@Christinadobrzyn Christinadobrzyn added the Needs Reproduction Reproducible steps needed label May 16, 2023
@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented May 16, 2023

I have not been able to reproduce this bug yet. Asking QA for a little help as in the video it looks like they are using a lot of different profiles. https://expensify.slack.com/archives/C9YU7BX5M/p1684257572010619

@kbecciv
Copy link
Author

kbecciv commented May 17, 2023

Issue is no longer reproduced

Screen.Recording.2023-05-17.at.20.19.39.mov

@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented May 17, 2023

Thanks @kbecciv Going to close it since it can't be reproduced. Removed Upwork job posting

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 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Needs Reproduction Reproducible steps needed
Projects
None yet
Development

No branches or pull requests

6 participants