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

[HOLD for payment 2024-04-05] Expense - Expense preview shows skeleton loading after paying with Expensify #38869

Closed
6 tasks done
lanitochka17 opened this issue Mar 23, 2024 · 17 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering Weekly KSv2

Comments

@lanitochka17
Copy link

lanitochka17 commented Mar 23, 2024

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


Version Number: 1.4.56-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: around Reproducible in production?:
Issue reported by: Applause - Internal Team

Action Performed:

Precondition:

  • Admin and member are in the same workspace
  • The workspace has a bank account set up
  • Member does not have any bank account
  1. Go to staging.new.expensify.com
  2. [Member] Request money from the workspace chat
  3. [Admin] Go to workspace chat with member
  4. [Admin] Click on the expense preview
  5. [Admin] Click Pay with Expensify
  6. [Admin] Go back to 1:1 DM and return to expense preview to see the skeleton loading if needed

Expected Result:

In Step 5, after clicking Pay with Expensify, the page will not be blank or show skeleton loading

Actual Result:

In Step 5, after clicking Pay with Expensify, the page shows skeleton loading above the "settling up" message

Workaround:

Unknown

Platforms:

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

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6423773_1711143706448.20240323_053656.mp4

View all open jobs on GitHub

@lanitochka17 lanitochka17 added the DeployBlockerCash This issue or pull request should block deployment label Mar 23, 2024
Copy link
Contributor

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

Copy link

melvin-bot bot commented Mar 23, 2024

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

@lanitochka17
Copy link
Author

@luacmartins FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

@lanitochka17
Copy link
Author

We think that this bug might be related to #wave-collect - Release 1

@luacmartins
Copy link
Contributor

Looking at this one now

@luacmartins
Copy link
Contributor

This is caused by calling GetOlderActions in a loop

@luacmartins
Copy link
Contributor

@roryabraham
Copy link
Contributor

I'm not able to reproduce, but for an unrelated reason. I'm not able to get past the Expense report submission in a collect workspace. Going to try with fresh accounts + a free workspace

@roryabraham
Copy link
Contributor

I'm still struggling with this one - I reproduced it once but am running into two unrelated issues, reported here

@perunt
Copy link
Contributor

perunt commented Mar 26, 2024

I've added a test bank account and was wondering, can we use it to make payments?

I'm pretty sure the issue might be in getContinuousReportActionChain. It looks like after making a payment, this action either moves or changes the type of the reportAction.

@lanitochka17, could you help us out by sharing some logs? If you could add console.log('allReportActions', JSON.stringify(sortedReportActions)) in the right spot and send over the details, that would really help me pinpoint the issue.

@luacmartins
Copy link
Contributor

Hmm I was trying to get the logs asked above, but I can't reproduce the issue right now. Gonna keep trying.

@luacmartins
Copy link
Contributor

luacmartins commented Mar 26, 2024

Ok, managed to reproduce again. Here are the logs @perunt

allReportActions [{"actionName":"IOU","actorAccountID":2,"automatic":false,"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_3.png","isAttachment":false,"originalMessage":{"amount":75,"currency":"USD","IOUReportID":"1214606904707134","type":"pay","paymentType":"ACH"},"message":[{"html":"paid $0.75 with Expensify","text":"paid $0.75 with Expensify","isEdited":false,"type":"COMMENT"}],"person":[{"style":"strong","text":"cmartins@expensifail.com","type":"TEXT"}],"reportActionID":"3459719543132819571","shouldShow":true,"created":"2024-03-26 15:16:29.524","pendingAction":"add","whisperedToAccountIDs":[]},{"person":[{"type":"TEXT","style":"strong","text":"Concierge"}],"actorAccountID":2,"message":[{"type":"TEXT","style":"normal","text":"Reimbursement blocked: We're waiting for your company to update their business bank account."}],"originalMessage":{"automaticAction":true,"lastModified":"2024-03-25 20:31:31.595"},"avatar":"","created":"2024-03-25 20:31:31.595","timestamp":1711398691,"reportActionTimestamp":1711398691595,"automatic":true,"actionName":"OUTDATEDBANKACCOUNT","shouldShow":true,"reportActionID":"1827888261995192039","previousReportActionID":"3681167035629397809","lastModified":"2024-03-25 20:31:31.595","whisperedToAccountIDs":[]},{"person":[{"type":"TEXT","style":"strong","text":"cmartins@expensifail.com"}],"actorAccountID":2,"message":[{"type":"COMMENT","html":"approved $0.75","text":"approved $0.75","isEdited":false,"whisperedTo":[],"isDeletedParentAction":false,"deleted":"","reactions":[]}],"originalMessage":{"IOUReportID":1214606904707134,"amount":75,"currency":"USD","lastModified":"2024-03-25 20:31:31.487"},"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_3.png","created":"2024-03-25 20:31:31.487","timestamp":1711398691,"reportActionTimestamp":1711398691487,"automatic":false,"actionName":"APPROVED","shouldShow":true,"reportActionID":"3681167035629397809","previousReportActionID":"759212220634693227","lastModified":"2024-03-25 20:31:31.487","whisperedToAccountIDs":[]},{"person":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"}],"actorAccountID":705,"message":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"},{"type":"TEXT","style":"normal","text":" submitted this report"},{"type":"TEXT","style":"normal","text":" to "},{"type":"TEXT","style":"strong","text":"you"}],"originalMessage":{"cc":"","hasExternalComment":false,"lastModified":"2024-03-25 20:05:55.473","message":"","to":"cmartins@expensifail.com","workflow":"BASIC"},"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_10.png","created":"2024-03-25 20:05:55.473","timestamp":1711397155,"reportActionTimestamp":1711397155473,"automatic":false,"actionName":"SUBMITTED","shouldShow":true,"reportActionID":"759212220634693227","previousReportActionID":"4258134669701620864","lastModified":"2024-03-25 20:05:55.473","whisperedToAccountIDs":[]},{"person":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"}],"actorAccountID":705,"message":[{"type":"COMMENT","html":"CA$1.00 request","text":"CA$1.00 request","isEdited":false,"whisperedTo":[],"isDeletedParentAction":false,"deleted":"","reactions":[]}],"originalMessage":{"IOUReportID":1214606904707134,"IOUTransactionID":"7093758477670296768","amount":100,"comment":"","currency":"CAD","lastModified":"2024-03-25 20:05:52.945","participantAccountIDs":[705,0],"type":"create"},"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_10.png","created":"2024-03-25 20:05:52.945","timestamp":1711397152,"reportActionTimestamp":1711397152945,"automatic":false,"actionName":"IOU","shouldShow":true,"reportActionID":"4258134669701620864","previousReportActionID":"4154332254083280327","lastModified":"2024-03-25 20:05:52.945","whisperedToAccountIDs":[],"childReportID":7207016927163148,"childType":"chat","childReportNotificationPreference":"hidden"},{"reportActionID":"4154332254083280327","previousReportActionID":"0","actionName":"CREATED","created":"2024-03-25 20:05:52.942","reportActionTimestamp":1711397152942,"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_10.png","message":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"},{"type":"TEXT","style":"normal","text":" created this report"}],"person":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"}],"automatic":false,"sequenceNumber":0,"shouldShow":true,"lastModified":"2024-03-25 20:05:52.942"}]
allReportActions [{"actionName":"REIMBURSEMENTQUEUED","actorAccountID":2,"automatic":false,"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_3.png","created":"2024-03-26 15:16:30.139","isAttachment":false,"lastModified":"2024-03-26 15:16:30.139","message":[{"style":"strong","text":"You","type":"TEXT"},{"style":"normal","text":" queued this report for reimbursement (paid via bank account). Reimbursement is on hold until ","type":"TEXT"},{"style":"strong","text":"cmartins+2@expensifail.com","type":"TEXT"},{"style":"normal","text":" adds a bank account","type":"TEXT"}],"originalMessage":{"IOUReportID":1214606904707134,"amount":75,"bankAccountID":7,"fundID":0,"lastModified":"2024-03-26 15:16:30.139","originalCaller":"PayMoneyRequest","paymentMethod":"ach_debit","paymentType":"ACH","reimburserID":2,"reportActionID":3459719543132819500},"person":[{"style":"strong","text":"cmartins@expensifail.com","type":"TEXT"}],"previousReportActionID":"1827888261995192039","reportActionID":"6661925061748177534","reportActionTimestamp":1711466190139,"sequenceNumber":5,"shouldShow":true,"timestamp":1711466190,"whisperedToAccountIDs":[]},{"actionName":"IOU","actorAccountID":2,"automatic":false,"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_3.png","isAttachment":false,"originalMessage":{"amount":75,"currency":"USD","IOUReportID":"1214606904707134","type":"pay","paymentType":"ACH"},"message":[{"html":"paid $0.75 with Expensify","text":"paid $0.75 with Expensify","isEdited":false,"type":"COMMENT"}],"person":[{"style":"strong","text":"cmartins@expensifail.com","type":"TEXT"}],"reportActionID":"3459719543132819571","shouldShow":true,"created":"2024-03-26 15:16:29.524","whisperedToAccountIDs":[]},{"person":[{"type":"TEXT","style":"strong","text":"Concierge"}],"actorAccountID":2,"message":[{"type":"TEXT","style":"normal","text":"Reimbursement blocked: We're waiting for your company to update their business bank account."}],"originalMessage":{"automaticAction":true,"lastModified":"2024-03-25 20:31:31.595"},"avatar":"","created":"2024-03-25 20:31:31.595","timestamp":1711398691,"reportActionTimestamp":1711398691595,"automatic":true,"actionName":"OUTDATEDBANKACCOUNT","shouldShow":true,"reportActionID":"1827888261995192039","previousReportActionID":"3681167035629397809","lastModified":"2024-03-25 20:31:31.595","whisperedToAccountIDs":[]},{"person":[{"type":"TEXT","style":"strong","text":"cmartins@expensifail.com"}],"actorAccountID":2,"message":[{"type":"COMMENT","html":"approved $0.75","text":"approved $0.75","isEdited":false,"whisperedTo":[],"isDeletedParentAction":false,"deleted":"","reactions":[]}],"originalMessage":{"IOUReportID":1214606904707134,"amount":75,"currency":"USD","lastModified":"2024-03-25 20:31:31.487"},"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_3.png","created":"2024-03-25 20:31:31.487","timestamp":1711398691,"reportActionTimestamp":1711398691487,"automatic":false,"actionName":"APPROVED","shouldShow":true,"reportActionID":"3681167035629397809","previousReportActionID":"759212220634693227","lastModified":"2024-03-25 20:31:31.487","whisperedToAccountIDs":[]},{"person":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"}],"actorAccountID":705,"message":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"},{"type":"TEXT","style":"normal","text":" submitted this report"},{"type":"TEXT","style":"normal","text":" to "},{"type":"TEXT","style":"strong","text":"you"}],"originalMessage":{"cc":"","hasExternalComment":false,"lastModified":"2024-03-25 20:05:55.473","message":"","to":"cmartins@expensifail.com","workflow":"BASIC"},"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_10.png","created":"2024-03-25 20:05:55.473","timestamp":1711397155,"reportActionTimestamp":1711397155473,"automatic":false,"actionName":"SUBMITTED","shouldShow":true,"reportActionID":"759212220634693227","previousReportActionID":"4258134669701620864","lastModified":"2024-03-25 20:05:55.473","whisperedToAccountIDs":[]},{"person":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"}],"actorAccountID":705,"message":[{"type":"COMMENT","html":"CA$1.00 request","text":"CA$1.00 request","isEdited":false,"whisperedTo":[],"isDeletedParentAction":false,"deleted":"","reactions":[]}],"originalMessage":{"IOUReportID":1214606904707134,"IOUTransactionID":"7093758477670296768","amount":100,"comment":"","currency":"CAD","lastModified":"2024-03-25 20:05:52.945","participantAccountIDs":[705,0],"type":"create"},"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_10.png","created":"2024-03-25 20:05:52.945","timestamp":1711397152,"reportActionTimestamp":1711397152945,"automatic":false,"actionName":"IOU","shouldShow":true,"reportActionID":"4258134669701620864","previousReportActionID":"4154332254083280327","lastModified":"2024-03-25 20:05:52.945","whisperedToAccountIDs":[],"childReportID":7207016927163148,"childType":"chat","childReportNotificationPreference":"hidden"},{"reportActionID":"4154332254083280327","previousReportActionID":"0","actionName":"CREATED","created":"2024-03-25 20:05:52.942","reportActionTimestamp":1711397152942,"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_10.png","message":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"},{"type":"TEXT","style":"normal","text":" created this report"}],"person":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"}],"automatic":false,"sequenceNumber":0,"shouldShow":true,"lastModified":"2024-03-25 20:05:52.942"}]
allReportActions [{"actionName":"REIMBURSEMENTQUEUED","actorAccountID":2,"automatic":false,"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_3.png","created":"2024-03-26 15:16:30.139","isAttachment":false,"lastModified":"2024-03-26 15:16:30.139","message":[{"type":"TEXT","style":"strong","text":"You"},{"type":"TEXT","style":"normal","text":" queued this report for reimbursement (paid via bank account). Reimbursement is on hold until "},{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"},{"type":"TEXT","style":"normal","text":" adds a bank account"}],"originalMessage":{"IOUReportID":1214606904707134,"amount":75,"bankAccountID":7,"fundID":0,"lastModified":"2024-03-26 15:16:30.139","originalCaller":"PayMoneyRequest","paymentMethod":"ach_debit","paymentType":"ACH","reimburserID":2,"reportActionID":3459719543132819500},"person":[{"type":"TEXT","style":"strong","text":"cmartins@expensifail.com"}],"previousReportActionID":"1827888261995192039","reportActionID":"6661925061748177534","reportActionTimestamp":1711466190139,"sequenceNumber":5,"shouldShow":true,"timestamp":1711466190,"whisperedToAccountIDs":[],"actorEmail":"cmartins@expensifail.com"},{"actionName":"IOU","actorAccountID":2,"automatic":false,"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_3.png","isAttachment":false,"originalMessage":{"amount":75,"currency":"USD","IOUReportID":"1214606904707134","type":"pay","paymentType":"ACH"},"message":[{"html":"paid $0.75 with Expensify","text":"paid $0.75 with Expensify","isEdited":false,"type":"COMMENT"}],"person":[{"style":"strong","text":"cmartins@expensifail.com","type":"TEXT"}],"reportActionID":"3459719543132819571","shouldShow":true,"created":"2024-03-26 15:16:29.524","whisperedToAccountIDs":[]},{"person":[{"type":"TEXT","style":"strong","text":"Concierge"}],"actorAccountID":2,"message":[{"type":"TEXT","style":"normal","text":"Reimbursement blocked: We're waiting for your company to update their business bank account."}],"originalMessage":{"automaticAction":true,"lastModified":"2024-03-25 20:31:31.595"},"avatar":"","created":"2024-03-25 20:31:31.595","timestamp":1711398691,"reportActionTimestamp":1711398691595,"automatic":true,"actionName":"OUTDATEDBANKACCOUNT","shouldShow":true,"reportActionID":"1827888261995192039","previousReportActionID":"3681167035629397809","lastModified":"2024-03-25 20:31:31.595","whisperedToAccountIDs":[],"actorEmail":"cmartins@expensifail.com"},{"person":[{"type":"TEXT","style":"strong","text":"cmartins@expensifail.com"}],"actorAccountID":2,"message":[{"type":"COMMENT","html":"approved $0.75","text":"approved $0.75","isEdited":false,"whisperedTo":[],"isDeletedParentAction":false,"deleted":"","reactions":[]}],"originalMessage":{"IOUReportID":1214606904707134,"amount":75,"currency":"USD","lastModified":"2024-03-25 20:31:31.487"},"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_3.png","created":"2024-03-25 20:31:31.487","timestamp":1711398691,"reportActionTimestamp":1711398691487,"automatic":false,"actionName":"APPROVED","shouldShow":true,"reportActionID":"3681167035629397809","previousReportActionID":"759212220634693227","lastModified":"2024-03-25 20:31:31.487","whisperedToAccountIDs":[],"actorEmail":"cmartins@expensifail.com"},{"person":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"}],"actorAccountID":705,"message":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"},{"type":"TEXT","style":"normal","text":" submitted this report"},{"type":"TEXT","style":"normal","text":" to "},{"type":"TEXT","style":"strong","text":"you"}],"originalMessage":{"cc":"","hasExternalComment":false,"lastModified":"2024-03-25 20:05:55.473","message":"","to":"cmartins@expensifail.com","workflow":"BASIC"},"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_10.png","created":"2024-03-25 20:05:55.473","timestamp":1711397155,"reportActionTimestamp":1711397155473,"automatic":false,"actionName":"SUBMITTED","shouldShow":true,"reportActionID":"759212220634693227","previousReportActionID":"4258134669701620864","lastModified":"2024-03-25 20:05:55.473","whisperedToAccountIDs":[],"actorEmail":"cmartins+2@expensifail.com"},{"person":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"}],"actorAccountID":705,"message":[{"type":"COMMENT","html":"CA$1.00 request","text":"CA$1.00 request","isEdited":false,"whisperedTo":[],"isDeletedParentAction":false,"deleted":"","reactions":[]}],"originalMessage":{"IOUReportID":1214606904707134,"IOUTransactionID":"7093758477670296768","amount":100,"comment":"","currency":"CAD","lastModified":"2024-03-25 20:05:52.945","participantAccountIDs":[705,0],"type":"create"},"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_10.png","created":"2024-03-25 20:05:52.945","timestamp":1711397152,"reportActionTimestamp":1711397152945,"automatic":false,"actionName":"IOU","shouldShow":true,"reportActionID":"4258134669701620864","previousReportActionID":"4154332254083280327","lastModified":"2024-03-25 20:05:52.945","whisperedToAccountIDs":[],"childReportID":7207016927163148,"childType":"chat","childReportNotificationPreference":"hidden","actorEmail":"cmartins+2@expensifail.com"},{"reportActionID":"4154332254083280327","previousReportActionID":"0","actionName":"CREATED","created":"2024-03-25 20:05:52.942","reportActionTimestamp":1711397152942,"avatar":"https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_10.png","message":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"},{"type":"TEXT","style":"normal","text":" created this report"}],"person":[{"type":"TEXT","style":"strong","text":"cmartins+2@expensifail.com"}],"automatic":false,"sequenceNumber":0,"shouldShow":true,"lastModified":"2024-03-25 20:05:52.942"}]

@luacmartins
Copy link
Contributor

I've added a test bank account and was wondering, can we use it to make payments?

You could, but you'd have funds debited from it, but as long as you're paying me I'm cool with that 😄

I'm pretty sure the issue might be in getContinuousReportActionChain. It looks like after making a payment, this action either moves or changes the type of the reportAction.

Yes, that's where I've traced it down to as well. I'm not too sure what's wrong with it yet, but maybe the logs above might help.

@luacmartins luacmartins mentioned this issue Mar 26, 2024
50 tasks
@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Hourly KSv2 Weekly KSv2 labels Mar 27, 2024
@yuwenmemon yuwenmemon removed the DeployBlockerCash This issue or pull request should block deployment label Mar 27, 2024
@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Mar 29, 2024
@melvin-bot melvin-bot bot changed the title Expense - Expense preview shows skeleton loading after paying with Expensify [HOLD for payment 2024-04-05] Expense - Expense preview shows skeleton loading after paying with Expensify Mar 29, 2024
Copy link

melvin-bot bot commented Mar 29, 2024

Reviewing label has been removed, please complete the "BugZero Checklist".

@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Mar 29, 2024
Copy link

melvin-bot bot commented Mar 29, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.57-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

If no regressions arise, payment will be issued on 2024-04-05. 🎊

Copy link

melvin-bot bot commented Apr 5, 2024

Skipping the payment summary for this issue since all the assignees are employees or vendors. If this is incorrect, please manually add the payment summary SO.

@luacmartins
Copy link
Contributor

This is done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering Weekly KSv2
Projects
None yet
Development

No branches or pull requests

5 participants