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

Fetch more messages by scrolling #308

Open
emanueljg opened this issue Jun 5, 2023 · 5 comments
Open

Fetch more messages by scrolling #308

emanueljg opened this issue Jun 5, 2023 · 5 comments
Labels
feat New feature or request

Comments

@emanueljg
Copy link

emanueljg commented Jun 5, 2023

Currently when selecting a channel and scrolling through its history, there is no way of fetching messages beyond the messages_limit messages loaded in at the time of selection. A way of querying for older messages, either implicitly by trying to scroll or an explicit keypress to trigger it would be nice. The discord API does allow message fetching with an offset so those are some good news.

A temporary fix for now is obviously to just increase the messages limit, but this of course is suboptimal when you really only want to preview few messages as you're now stuck with slower channel loading times with no real benefit.

@emanueljg emanueljg changed the title Allow fetching more messages by scrolling Fetch more messages by scrolling Jun 5, 2023
@craftxbox
Copy link

It's worth nothing: I have heard talk that discord uses /messages with offset for detecting illegal API usage. It may be worth testing the safety of using it in our case here.

@emanueljg
Copy link
Author

It's worth nothing: I have heard talk that discord uses /messages with offset for detecting illegal API usage. It may be worth testing the safety of using it in our case here.

Thank you for this info. Do you know if discord bans people account-wise or IP-wise? If the former then people that likes to live on the edge could try it out first. We could make the feature toggleable.

@craftxbox
Copy link

It's worth nothing: I have heard talk that discord uses /messages with offset for detecting illegal API usage. It may be worth testing the safety of using it in our case here.

Thank you for this info. Do you know if discord bans people account-wise or IP-wise? If the former then people that likes to live on the edge could try it out first. We could make the feature toggleable.

I'm not certain. I know server bans are executed by account and IP but I'm unaware about the specifics of platform bans. As long as you haven't been doing anything malicious, TNS usually grants amnesty for an API-abuse ban, but the issue is getting ahold of them before the account autodeletes.

@ghost
Copy link

ghost commented Jun 6, 2023

For testing, use an alt account and a VPN so you could hide your IP.

@ayn2op ayn2op added the feat New feature or request label Nov 1, 2023
@diamondburned
Copy link

Hi, Dissent developer here! There shouldn't be any issue as far as API usage goes, just make sure your code isn't buggy that it spams the API. Dissent itself has this feature.

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

No branches or pull requests

4 participants