akshayasalvi
akshayasalvi
@tgolen @mananjadhav I have made additional changes so that the `:emojicode:` searching logic works as expected. Also there is one test failing because we're checking if the emoji matches a...
@tgolen @mananjadhav The new emojis might not work with the existing regex, and we will have to update the `isSingleEmoji` function. Take the following example of `"face_in_clouds:` emoji. The emoji...
@tgolen @mananjadhav Apologies for too many messages. I finally found the problem. The tests were failing due to the following reasons: - `isSingleEmoji` wasn't comprehensive enough to cover the composite...
> > * There was one fix required for male/female emojis to work with `containsOnlyEmojis` > > Can you highlight what you mean by the fix here? @mananjadhav All the...
I accidentally removed @mananjadhav from the review, by request a second review from @tgolen
@mananjadhav PR updated