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

Chat - Skeleton view is not displayed above the IOU after request money, console error is showing with Report no longer exists. #13679

Closed
kbecciv opened this issue Dec 17, 2022 · 4 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff

Comments

@kbecciv
Copy link

kbecciv commented Dec 17, 2022

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


Issue found when executing PR #13546

Action Performed:

  1. Go to staging.new.expensify.com
  2. Log in on an account that has a DM chat with many actions
  3. Navigate to a chat that is not the aforementioned chat with many actions
  4. Log out, so that you clear Onyx
  5. Log in.
  6. Make a request money from the person you're chatting with on the DM with many actions

Expected Result:

Skeleton view is displayed above it after request money, and no console error

Actual Result:

Skeleton view is not displayed above it after request money, console error

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • Web
  • Mobile Web

Version Number: 1.2.41.1

Reproducible in staging?: Yes

Reproducible in production?: No (new feature)

Email or phone of affected tester (no customers):

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

Notes/Photos/Videos: Any additional supporting documentation

13590.Web.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

@kbecciv kbecciv added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Dec 17, 2022
@melvin-bot melvin-bot bot locked and limited conversation to collaborators Dec 17, 2022
@melvin-bot melvin-bot bot added the Overdue label Dec 19, 2022
@alexpensify
Copy link
Contributor

Not overdue, I was OOO on Friday.

@melvin-bot melvin-bot bot removed the Overdue label Dec 19, 2022
@yuwenmemon
Copy link
Contributor

@kbecciv I don't believe you're following the test steps correctly - can you make sure the chat with many messages is not clicked on before you make the money request?

@yuwenmemon yuwenmemon self-assigned this Dec 19, 2022
@yuwenmemon yuwenmemon added the Internal Requires API changes or must be handled by Expensify staff label Dec 19, 2022
@kbecciv
Copy link
Author

kbecciv commented Dec 19, 2022

@yuwenmemon Just re-test the PR, it Pass

13590.Web-retest.mp4

@yuwenmemon
Copy link
Contributor

Thanks! Will close this out then.

@Expensify Expensify unlocked this conversation Mar 21, 2023
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 Internal Requires API changes or must be handled by Expensify staff
Projects
None yet
Development

No branches or pull requests

3 participants