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

Links in message text #43

Open
Farzat07 opened this issue Dec 16, 2021 · 4 comments · May be fixed by #50
Open

Links in message text #43

Farzat07 opened this issue Dec 16, 2021 · 4 comments · May be fixed by #50

Comments

@Farzat07
Copy link
Contributor

Sometimes a part of the message text hides a link to another message or even an external website. For example, the TelegramTips channel is full of such links. However, when the channel is archived and built into a website, these links are shown as regular text.

I am not sure whether the data is saved into the database but not included in the html, or the data is not even saved in the database in the first place. If it is saved in the database, I would appreciate if the relevant field is specified, so that I can include it in the template.

@knadh
Copy link
Owner

knadh commented Dec 18, 2021

Could you please post screenshots demonstrating this?

@faraazb
Copy link
Contributor

faraazb commented Feb 3, 2022

Basically, all formatting such as bold, italic, inline links, etc. is removed (because message.raw_text is being used here). message.text contains the message content as sanitized HTML (with client.parse_mode = 'html') which can be rendered directly. Can I make a PR?

@knadh
Copy link
Owner

knadh commented Feb 4, 2022

Yep, makes sense.

@faraazb faraazb linked a pull request Feb 5, 2022 that will close this issue
@aizelauna
Copy link

I am trying this very interesting tool, thanks a lot for this great work!
I encountered this issue on a channel where every message point toward an external website.
It would be really great if the PR could be merged. Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants