[WIP][BREAKING CHANGE] api: Update to Layer 108#5
[WIP][BREAKING CHANGE] api: Update to Layer 108#5aarani wants to merge 12 commits intonblockchain:masterfrom
Conversation
|
Needs Testing. |
|
@aarani good work! @CheshireCaat hey you might be interested in this, can you help us test it? @aarani : by the way, as far as I understand, this PR has 2 parts: one which you updated manually, and the other one which you updated automatically via the generator, can you explain how did you do the latter? |
Our generators is a little idiot that cause it's generated code to be unusable without manual modifications. for now before that we can fix it all generated codes need to be formatted and it's errors must be fixed manually and as this layer has a lot of changes it breaks the TelegramClient wrapper which I had to fix too |
|
@aarani I see, but still, how did you call the generator? |
Nothing crazy, just run "TLGenerator <path_to_json_schema>" |
And where to get the json schema? |
Telegram started updating their documentation and they now have the schema json on their website before that you had to extract it from webogram config files |
|
are you sure? check the PR 906 of TLSharp |
of course. afaik there is no need to update anything else to use new layers. we need to support mtproto 2.0 but that's story for another day for now just updating the api is enough |
|
are you saying that the current schema in https://core.telegram.org/schema/json is only MTProto2.0 compatible and TLSharp/TgSharp is not MTProto2.0 compatible yet? |
I'm exactly saying the opposite. It's compatible with both MTProto versions and using it doesn't need any other changes except changes to schema and models |
what changes to schema and models? |
changes that I did inside this pull request |
|
You mean the changes after running the generator? |
Yup |
|
So then the exact instructions are:
right? |
and fix all the errors that TLGenerator made and change all the legacy methods that are removed from previous layers that will cause TelegramClient.cs to break and change TelegramClient.cs#83 to let telegram know what version of api we're using to communicate |
I was asking how to just run the generator, not how to fix the mess later (that was already clear). |
I was just describing the protocol, sure you're welcome |
|
@aarani do you mind fixing the conflicts, and adding this to the README file: #5 (comment) |
|
Telegram requests to change DC, the software tries to do it -->
|
|
this is happening with my Emirati number, i can give it to you for testing purposes |
|
@aarani how can a layer update cause transport/connection issues? |
d1297ff to
b5ca41f
Compare
|
@knocte Please push this already and just create a legacy branch for 2FA Users |
That will be on roadmap as soon as this pr get merged, feel free to create a pr for it |
Well, let's consider TLSharp the legacy branch? I'm not doing fixes to that repo anymore. |
I'm not comfortable pushing this if the generator is not in sync with what is being generated here, I'm trying to understand why, first. |
Have fun |
13c61a3 to
605f368
Compare
This was the result of running the latest version of the generator, which fixes many whitespace issues such as trailing whitespace.
Manual edits needed after code autogeneration.
For consistency.
|
Superseded by #13 , sorry for the long delay! |
|
How I can login after registration? I want login after close my windows app. By which method I can check in session file? this is my code |
No description provided.