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.57k stars 2.91k forks source link

[Backwards/forwards compatibility] Unmigrated reports are displayed tapping on expense in search result #50968

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?: 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: @quinthar Slack conversation: https://expensify.slack.com/archives/C05LX9D6E07/p1729061278156169

Action Performed:

  1. Log in to account
  2. Click on search from the bottom menu

Expected Result:

Only migrated reports should be displayed

Actual Result:

Non migrated weired expenses are displayed

Workaround:

unknown

Platforms:

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

Screenshots/Videos

Screenshot_20241016-144458

Add any screenshot/video evidence

View all open jobs on GitHub

Issue OwnerCurrent Issue Owner: @muttmuure
Issue OwnerCurrent Issue Owner: @muttmuure
melvin-bot[bot] commented 1 month ago

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

melvin-bot[bot] commented 1 month ago

Triggered auto assignment to @Julesssss (AutoAssignerNewDotQuality)

MelvinBot commented 1 month ago

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

Julesssss commented 1 month ago

Waiting on reproduction

muttmuure commented 1 month ago

@luacmartins will you fit this one into your cycles?

luacmartins commented 1 month ago

Assigning myself since it's related to Search

muttmuure commented 3 weeks ago

I think @luacmartins is pretty busy right now so I'm going to call this out as a hot pick

mvtglobally commented 3 weeks ago

Issue not reproducible during KI retests. (First week)

luacmartins commented 3 weeks ago

This one will be tricky to reproduce since it's likely an edge case from old policies. We'd need more data on the policy to know. We tried to get it in the Slack thread, but we could never find the policy linked to it. Maybe we can run a CQ to do so:

SELECT JSON(value) FROM nameValuePairs WHERE name = 'expensify_policyB40E8D400EDDA001';
luacmartins commented 3 weeks ago

Requested the CQ here - https://github.com/Expensify/Expensify/issues/440142

melvin-bot[bot] commented 3 weeks ago

@muttmuure 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!

muttmuure commented 1 week ago

Not overdue, still considering prioritization