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

Improve history handling #97

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Improve history handling #97

wants to merge 17 commits into from

Conversation

majn
Copy link
Contributor

@majn majn commented Mar 23, 2016

Allow fetching message history between max_id and min_id in a new api call. This is needed for fetching channel history when the amount of required amount of messages is unknown.

Additionally allow fetching more than 100 channel messages. I don't know why it was disabled in the first place, but I tested it thoroughly and the history function doesn't have any issue handling the channel return type CODE_messages_channel_messages.

A last change is to use less confusing naming in get_history_extra struct, offset_id is now actually called offset_id except of max_id.

majn and others added 11 commits March 3, 2016 02:36
Add a new function for fetching history ranges between a max and
a min id. This can come handy for fetching history deltas in
supergroups on login when only the min_id is known.
Fix reported by @spaeps in majn/telegram-purple#238
Closes vysheng#42
I didn't actually do anything, except forward this patch.
The referenced target "gen-mime-types" was wrong anyway.
./auto-static-autocomplete.c:72:1: error: control reaches end of
non-void function [-Werror=return-type]
@nima1363azad
Copy link

dh_gen_ok#3bcbf734 nonce:int128 server_nonce:int128 new_nonce_hash1:int128 = Set_client_DH_params_answer;
dh_gen_retry#46dc1fb9 nonce:int128 server_nonce:int128 new_nonce_hash2:int128 = Set_client_DH_params_answer;
dh_gen_fail#a69dae02 nonce:int128 server_nonce:int128 new_nonce_hash3:int128 = Set_client_DH_params_answer;

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.

5 participants