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

[TelegramBridge] Display the name of the attachments #2003

Merged
merged 4 commits into from
Mar 16, 2021

Conversation

drego85
Copy link
Contributor

@drego85 drego85 commented Mar 2, 2021

Sometimes attachments are posted in Telegram channels without any text.

The script recognizes a new message but does not report any text, with this commit the file names will also be included in the RSS Feed.

Sometimes attachments are posted in Telegram channels without any text. 

The script recognizes a new message but does not report any text, with this commit the file names will also be included in the RSS Feed.
@drego85
Copy link
Contributor Author

drego85 commented Mar 2, 2021

Before:

Schermata 2021-03-02 alle 15 45 15

After:

Schermata 2021-03-02 alle 15 45 18

Copy link
Contributor

@VerifiedJoseph VerifiedJoseph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this.

Please update the pull request's title to comply with the pull request policy (it should start with [TelegramBridge]) and fix the issues below to comply with the coding style policy.

bridges/TelegramBridge.php Outdated Show resolved Hide resolved
bridges/TelegramBridge.php Outdated Show resolved Hide resolved
bridges/TelegramBridge.php Outdated Show resolved Hide resolved
@drego85 drego85 changed the title Display the name of the attachments [TelegramBridge] Display the name of the attachments Mar 2, 2021
drego85 and others added 3 commits March 2, 2021 21:45
Co-authored-by: Joseph <joseph@verifiedjoseph.com>
Co-authored-by: Joseph <joseph@verifiedjoseph.com>
Co-authored-by: Joseph <joseph@verifiedjoseph.com>
@drego85
Copy link
Contributor Author

drego85 commented Mar 2, 2021

Thanks @VerifiedJoseph ;) updated!

@em92
Copy link
Contributor

em92 commented Mar 16, 2021

Hi, @drego85 ! Could you show me example of channel with those attachments?

@drego85
Copy link
Contributor Author

drego85 commented Mar 16, 2021

You can use my personal test channel: https://t.me/TestRSSBridge

@em92 em92 merged commit 3aafd44 into RSS-Bridge:master Mar 16, 2021
@em92
Copy link
Contributor

em92 commented Mar 16, 2021

gj!

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 this pull request may close these issues.

3 participants