Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle emojis #6

Closed
samadii opened this issue Jun 13, 2021 · 4 comments
Closed

Handle emojis #6

samadii opened this issue Jun 13, 2021 · 4 comments
Labels
bug Something isn't working

Comments

@samadii
Copy link

samadii commented Jun 13, 2021

If a URL has 'https://' at the first, it will be replaced with 'http://ttps//' in the cleaned one !!

And If a URL has 'http://' at the first, it will be replaced with 'http://ttp//' in the cleaned one !!

Please fix it.

@GauthamramRavichandran
Copy link
Owner

Could you provide an example or attach a screenshot of the said bug? Bot is working fine with both URL types you mentioned.

@samadii
Copy link
Author

samadii commented Jun 13, 2021

Oh :) i found the reason.
It will happen only when there is an emoji in the text before URL, and not a problem at all
Screenshot_2021-06-13-14-41-30

@GauthamramRavichandran GauthamramRavichandran changed the title A bug Handle emojis Jun 13, 2021
@GauthamramRavichandran GauthamramRavichandran added the bug Something isn't working label Jun 13, 2021
@GauthamramRavichandran
Copy link
Owner

Thank you for your screenshot, will try to handle emojis

@GauthamramRavichandran
Copy link
Owner

Fixed with recent commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants