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] Disable limit to the number of parsed articles #2084

Open
squromiv opened this issue Apr 26, 2021 · 7 comments · May be fixed by #3874
Open

[TelegramBridge] Disable limit to the number of parsed articles #2084

squromiv opened this issue Apr 26, 2021 · 7 comments · May be fixed by #3874
Labels
Feature-Request Issue is a feature request

Comments

@squromiv
Copy link

Often I need to get not only the last articles from a channel, but also previous 100-200-500 and more messages. It will be great to have an option to regulate limit of messages to grab or to turn off this limit (best solution for me).

@squromiv squromiv added the Feature-Request Issue is a feature request label Apr 26, 2021
@t0stiman
Copy link
Contributor

The bridge works by downloading https://t.me/s/<channel_name>. That page only shows the latest 20 posts in the channel (unless you scroll up, which the bridge can't do), which is why TelegramBridge only returns the latest 20 posts.
To get more then 20 posts we would need to fetch them from elsewhere I think. I'm gonna leave this issue for someone else to implement.

@em92
Copy link
Contributor

em92 commented Jul 29, 2021

Someone asked me for this, I implemented this for money.
Here is the modified bridge.
https://github.com/em92/rss-bridge/blob/58746t/bridges/TelegramBridge.php

Also that modified bridge somehow solves #1901

@t0stiman
Copy link
Contributor

t0stiman commented Jul 30, 2021

Okay cool, so just yeet that into a pull request?

@em92
Copy link
Contributor

em92 commented Jul 30, 2021

No, I won't accept this feature. Bridge with this feature will generate a lot of traffic to telegram and possibly will lead to ban.

@dumprop
Copy link

dumprop commented Aug 16, 2021

No, I won't accept this feature. Bridge with this feature will generate a lot of traffic to telegram and possibly will lead to ban.

Может добавить его как отдельный бридж? Чтобы каждый мог при необходимости использовать это из коробки (думаю такие запросы будут повторяться тк не все смогут найти этот тикет)
А хостеры публичных бриджей смогут выключать это чтобы не получить бан

@dumprop
Copy link

dumprop commented Aug 16, 2021

The bridge works by downloading https://t.me/s/<channel_name>. That page only shows the latest 20 posts in the channel (unless you scroll up, which the bridge can't do), which is why TelegramBridge only returns the latest 20 posts.
To get more then 20 posts we would need to fetch them from elsewhere I think. I'm gonna leave this issue for someone else to implement.

It doesn't need to scroll up
https://t.me/s/telegram?before=20 and you get first 20 posts, https://t.me/s/telegram?before=40 and you get 21-40 posts, etc
There no too much channels with 1k+ posts
I think it will be cool feature out of the box
But yes, it's probably a headache for public instances, some possible restrictions are needed
@em92 what do you think about that? seems you have another implementation

@dvikan
Copy link
Contributor

dvikan commented Sep 12, 2023

this is still interesting. those before params still works

@n-kam n-kam linked a pull request Jan 2, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature-Request Issue is a feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants