Roji Philip
Roji Philip
> Yeah, I can't see the flickering anymore too, but you can see that it still shows the last message of the policy expense chat even though we set `forcePolicyNamePreview`...
> Based on the code, the expected result is to show the policy name as explained in my previous comment. You can revert #53293 and see it will show the...
@mkzie2 Thanks for your proposal. The RCA is correct as we are showing the receipt preview without checking if we can show receipt preview or not. However, instead of focusing...
@mkzie2 @Krishna2323 Thanks both for your proposals. However, your proposals lack clarity on both the root cause and the solution and is unacceptable to me in its current form. Can...
## Proposal ### Please re-state the problem that we are trying to solve in this issue. We should not display empty receipt placeholder for split and invoice previews as there...
@mkzie2 @Krishna2323 Since your proposals are still unacceptable to me in it’s current form, I have put forward a proposal to avoid further delays here. Let me know if you...
> What is the difference between my proposal and your proposal here @mkzie2 Maybe I am missing something here. Can you please share a test video based on your proposal?...
> I'm not sure what's missing in [my proposal](https://github.com/Expensify/App/issues/53524#issuecomment-2517580435), It uses the same logic as applied in `MoneyRequestView`, do you need a test branch or something? @Krishna2323 Why is there...
Thanks @mkzie2 @Krishna2323 for the follow-up with your proposals. I agree @mkzie2 that even split expenses can have receipts and we should show the receipt if present. But @Krishna2323 proposal...
@m-natarajan Is this still reproducible? I cannot https://github.com/user-attachments/assets/807dac6a-4756-4e44-b890-4c8e03c27cca