Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.12k stars 2.62k forks source link

[$250] Transaction gets messed up when a scanned receipt merges with imported transaction #42946

Open m-natarajan opened 1 month ago

m-natarajan commented 1 month ago

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: Reproducible in staging?: needs reproduction Reproducible in production?: needs reproduction 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: @puneetlath Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1717101063221539

Action Performed:

  1. have a card importing in expensify
  2. buy something on that card
  3. scan the receipt for the expense
  4. wait until the transaction posts and gets imported into expensify

    Expected Result:

    Transaction shouldn't get messed up after receipt is merged

Actual Result:

Empty report is shown after scanning a receipt with imported transaction

Workaround:

unknown

Platforms:

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

Screenshots/Videos

image (11) image (10)

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01fb28bfad88e3ae68
  • Upwork Job ID: 1810699256766901689
  • Last Price Increase: 2024-07-09
Issue OwnerCurrent Issue Owner: @paultsimura
melvin-bot[bot] commented 1 month ago

Triggered auto assignment to @CortneyOfstad (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.

MelvinBot commented 1 month ago

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

CortneyOfstad commented 1 month ago

Will work on using my Expensify Card this week to import a transaction, but in the meantime, will put retest weekly on this to get it rolling.

melvin-bot[bot] commented 1 month ago

@CortneyOfstad Whoops! This issue is 2 days overdue. Let's get this updated quick!

CortneyOfstad commented 1 month ago

Cleaning lady gave me a list of supplies to purchase so will be testing the card by EOW!

CortneyOfstad commented 1 month ago

Items are being purchased today!

CortneyOfstad commented 4 weeks ago

Purchase has been made — just navigating ND to figure out how to see the imported ECard transaction in NewDot. It shows that it is set to import, but can't see the transactions in order to test.

CortneyOfstad commented 4 weeks ago

I am unable to find a way to test this appropriately in NewDot as the card settings/expenses are locked under OldDot.

Going to rely on QA to test and I am bumping them right now 👍

CortneyOfstad commented 4 weeks ago

Post to QA here

mvtglobally commented 4 weeks ago

QA team is not able to confirm if this issue is still reproducible during KI retests as it requires production transaction through a card along with automated card imports. This issue will need to be retested internally

CortneyOfstad commented 4 weeks ago

@puneetlath have you continued to experience this issue? I don't have any cards to import to test this unfortunately, so wanted to confirm.

melvin-bot[bot] commented 3 weeks ago

@CortneyOfstad this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

CortneyOfstad commented 3 weeks ago

AHA! I found a demo credit card account to test (https://stackoverflowteams.com/c/expensify/questions/67). Going to work on this today and hopefully recreate!

CortneyOfstad commented 3 weeks ago

Finally got the card and receipt ready to go and cannot test because I have the skeleton loader as I reported here

GH was created here

melvin-bot[bot] commented 2 weeks ago

@CortneyOfstad Eep! 4 days overdue now. Issues have feelings too...

CortneyOfstad commented 2 weeks ago

Not overdue — waiting for this to be retested!

CortneyOfstad commented 1 week ago

Unfortunately, I am still unable to test due to the infinite skeleton loader in the issue here. Bumped that issue in Slack to get some movement on it ASAP.

CortneyOfstad commented 1 week ago

Being proactive in commenting so Melvin does not raise the overdue flag. This is not overdue — I am working to find someone who can test this since I am still dealing with the skeleton loader

melvin-bot[bot] commented 3 days ago

@CortneyOfstad Eep! 4 days overdue now. Issues have feelings too...

CortneyOfstad commented 3 days ago

Alright, was finally able to get this tested and was able to recreate so getting eyes on this. I apologize for the massive delay on this, but opted for creating a new account instead of still waiting to have the skeleton loader corrected under my test account.

Getting eyes on this ASAP.

melvin-bot[bot] commented 3 days ago

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

melvin-bot[bot] commented 3 days ago

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

CortneyOfstad commented 3 days ago

VIP VSB is paused, so going to adjust the timing for this 👍