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

[Bug] Issue with chat ls for chat with more then 101 topics #546

Open
AlphaZ19 opened this issue Mar 21, 2024 · 4 comments · May be fixed by #556
Open

[Bug] Issue with chat ls for chat with more then 101 topics #546

AlphaZ19 opened this issue Mar 21, 2024 · 4 comments · May be fixed by #556
Assignees
Labels
bug Something isn't working

Comments

@AlphaZ19
Copy link

Describe the bug

For chat with more then 101 Topics, tdl chat ls only shows 101 topics, the rest are hidden.

To Reproduce

you can try it on chat https://t.me/haijiaovideo, which ID is 1943412408

Expected behavior

It should show all the topics, if too much for tdl chat ls, maybe you can make a tdl topic ls -c CHAT something like this.

Version

Version: 0.16.1
Commit: bb2c69b
Date: 2024-03-01T13:21:31Z

go1.21.7 windows/amd64

Which OS are you running tdl on?

Windows

Additional context

No response

@AlphaZ19 AlphaZ19 added the bug Something isn't working label Mar 21, 2024
@pgotsis
Copy link

pgotsis commented Mar 22, 2024

I am facing the same problem. I have checked both table and json format. Both of them do not go over the most recently updated 101 topics.

@pgotsis
Copy link

pgotsis commented Mar 22, 2024

I believe that the limitation is here:

https://github.com/iyear/tdl/blob/master/app/chat/ls.go#L231

@pgotsis
Copy link

pgotsis commented Mar 22, 2024

I have looked at it a bit more and it seems that the problem is because of the ChannelsGetForumTopics. I am not 100% sure, but this API function returns topics in batches, expecting the client code to handle requesting more batches until all items have been received. I base my assumption on testing a change in line 231 (which did not work) and this discussion (https://stackoverflow.com/questions/77538243/retrieve-the-forum-topics-of-a-channel-group-in-telegram-using-wtelegramclient-a).

@XMLHexagram
Copy link
Contributor

hello @iyear, I believe PR #556 fix this issue. Would you have spare time to review that PR.

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

Successfully merging a pull request may close this issue.

4 participants