Aimane Chnaif

Results 118 comments of Aimane Chnaif

All 3 proposals above work but depending on design choice, we can pick one up. @shawnborton can you please help review screenshots in proposals and provide feedback? 1. keep 1...

Thanks @shawnborton @thienlnam let's go with @huzaifa-99's [proposal](https://github.com/Expensify/App/issues/17631#issuecomment-1515886277) as it's the most close. 🎀 👀 🎀 C+ reviewed

> @aimane-chnaif do we need a regression test for this one? No PR caused regression. The issue was from library (actually not bug). Since this is minor style polish and...

I reproduced. I think this should be fixed in backend. Here's the root cause with example: report data: `reportActions` associated with this report doesn't exist before opening report so it...

> @aimane-chnaif Quick question: Does the bug above gets fixed if we refresh the page or does it persist? It persists as long as that report is not opened. (refresh...

> I am thinking maybe frontend can handle this differently i.e. rather than using the `text` content for LHN, they can parse the HTML content following this That makes sense....

Seems like we're holding for https://github.com/Expensify/App/issues/34175

https://github.com/Expensify/App/issues/34175 was closed

I don't see the need of waiting for https://github.com/Expensify/App/issues/35626 (seems like taking a while to fix) while the behavior is weird which requires urgent fix and the solution is straightforward.

@abzokhattab [proposed](https://github.com/Expensify/App/issues/35969#issuecomment-1930809598) first. Let's get engineer's thoughts whether to fix or hold. 🎀 👀 🎀 C+ reviewed