Pujan Shah

Results 78 comments of Pujan Shah

We can proceed with @bernhardoj's proposal to filter actions to consider visible actions only to derive hasNewestReportAction. 🎀👀🎀 C+ reviewed

# BugZero Checklist: - [X] **[Contributor]** Classify the bug: Bug classification Source of bug: - [ ] 1a. Result of the original design (eg. a case wasn't considered) - [X]...

@RachCHopkins @johnmlee101 @mananjadhav Do you think we need to handle other similar issues in this single [PR](https://github.com/Expensify/App/pull/54409)? https://github.com/Expensify/App/issues/53842 https://github.com/Expensify/App/issues/54637 https://github.com/Expensify/App/issues/53779

Thanks for the proposals. @FitseTLT and @prakashbask's proposal of regex new instance won't solve the problem as it will only make it consistent(isAnchorTag gets true) and paste will be always...

@FitseTLT The problem isn't only making `isAnchorTag` consistent, it also should paste correct text without the link when the user chooses "Paste as plain text".

> You have to first understand the purposes of isAnchorTag check please refer to https://github.com/Expensify/App/pull/42147 for more context where they have implemented to make the copy of anchor to be...

> When choosing "Paste as plain text", the message is pasted without any hyperlink format @alexpensify Could you plz confirm the expected behavior for this issue as @FitseTLT is requesting?...

> @Pujan92 @alexpensify We will need to club this [53832](https://github.com/Expensify/App/issues/53832) issue also to discuss the expected behaviour Agree @prakashbask, both can be clubbed. The only platform that differs is Android...

Setting the plainText as htmlText seems logically wrong to me, I think instead the react native android should support the text/html if the passed html string worked with type text/plain.

Setting the plainText as htmlText in the clipboard seems logically wrong to me, but this only looks like the option(setting markdown text in text/plain option) for android. As I am...