Eric Han

Results 118 comments of Eric Han

@bernhardoj 's [proposal](https://github.com/Expensify/App/issues/39647#issuecomment-2039767594) looks good to me. I agree with their RCA and their solution. I also agree we should fix other money request flows together. 🎀👀🎀 C+ reviewed

@kadiealexander Based on the discussions above, can you help to update the `Expected results` section of this issue?

> @kadiealexander Based on the discussions above, can you help to update the `Expected results` section of this issue? @kadiealexander Friendly bump!

Or @bernhardoj would you like to update your proposal according to @aldo-expensify 's https://github.com/Expensify/App/issues/39647#issuecomment-2057456493 ?

@bernhardoj Thanks for the update! Based on the above discussions, yes, it makes more sense to `Don't show the display name when the lastActorAccountID is 0`. So, @bernhardoj 's [alternative...

@aldo-expensify Should we assign @bernhardoj to move this issue forward?

@aldo-expensify Thanks for driving clarity. @bernhardoj Can you take some time to update proposal based on the expected result^ ?

@bernhardoj Thanks for the update. Yeah, the 2nd alternative makes sense to me. @aldo-expensify All yours. 🎀👀🎀 C+ reviewed

Proposal We can support pasting HTML `p` tag and unordered list in `ExpensiMark.js` of [expensify-common](https://github.com/Expensify/expensify-common). Two changes will be added into `ExpensiMark.js` 1. Add space for unordered list item https://github.com/Expensify/expensify-common/blob/db2c245c1e6b39232092b1415b9235fadda2a529/lib/ExpensiMark.js#L226-L231...

> @eh2077 Not working for me, can you check and update please thanks! Hi @Santhosh-Sellavel, thanks for checking the proposal! I apply the diff of upstream lib `expensify-common` to App...