Roji Philip

Results 105 comments of 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