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.58k stars 2.92k forks source link

[Search v2.5] Transactions without MoneyRequest preview actions are shown as not found when user tries to open them #52074

Open luacmartins opened 3 weeks ago

luacmartins commented 3 weeks ago

Problem

Coming from this thread, some expenses on policy expense chat enabled policies don't have a money request preview action created for them, which means this code fails to create a transaction thread and the user sees a not found page.

Solution

Still discussing

Issue Owner

Current Issue Owner: @luacmartins

melvin-bot[bot] commented 3 weeks ago

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

isabelastisser commented 3 weeks ago

@luacmartins, does it need an internal or external assignment?

luacmartins commented 3 weeks ago

We're still discussing the expected behavior. I'll update the issue once we land on a solution since that'll determine if the work is internal or external.

melvin-bot[bot] commented 2 weeks ago

@luacmartins, @isabelastisser Huh... This is 4 days overdue. Who can take care of this?

isabelastisser commented 2 weeks ago

@luacmartins, any updates? Thanks!

luacmartins commented 2 weeks ago

Not yet, it seems like we're migrating our policies, so hopefully this should stop happening soon.

luacmartins commented 1 week ago

Prioritizing Search actions atm. I'll try to get to this next week.

isabelastisser commented 1 week ago

Not overdue. ^^

melvin-bot[bot] commented 1 week ago

@luacmartins @isabelastisser 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!

melvin-bot[bot] commented 1 week ago

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

luacmartins commented 1 week ago

Migration is underway. Waiting until that's done to retest this.

isabelastisser commented 5 days ago

Not overdue.

melvin-bot[bot] commented 3 days ago

@luacmartins, @isabelastisser Eep! 4 days overdue now. Issues have feelings too...

isabelastisser commented 2 days ago

Hi @luacmartins, any updates here? Thanks!

luacmartins commented 2 days ago

No updates yet. Moving this to weekly since it's an edge case