search

sochix / TLSharp

Telegram client library implemented in C#
1.01k stars 380 forks source link

updated link to the latest schema #906

Closed m-ehsan closed 4 years ago

m-ehsan commented 4 years ago

docs available at core.telegram.org are now pretty much up to date. (Layer 105) also the latest scheme can be found in official tdesktop repo.

CheshireCaat commented 4 years ago

Currently TeleSharp.Generator support only json schema to parse. Link to .tl format does not make sense. Maybe you can find official json schema or update the generator to support this format?

m-ehsan commented 4 years ago

Thanks, I didn't know about the generator. Now the link to the JSON schema is updated and it's pointing to the official Telegram website. (Currently at layer 105)

CheshireCaat commented 4 years ago

@m-ehsan Can you rebase your commits into one? @knocte what do you think about this?

knocte commented 4 years ago

docs available at core.telegram.org are now pretty much up to date. (Layer 105)

Shouldn't we update TLSharp to layer 105 first?

m-ehsan commented 4 years ago

@m-ehsan Can you rebase your commits into one?

Good idea.

docs available at core.telegram.org are now pretty much up to date. (Layer 105)

Shouldn't we update TLSharp to layer 105 first?

At least it can be a good source for feeding the code generator.

knocte commented 4 years ago

At least it can be a good source for feeding the code generator.

What do you mean?

m-ehsan commented 4 years ago

At least it can be a good source for feeding the code generator.

What do you mean?

I'm not really deep into TeleSharp.Generator but as far I know about this project, it gets JSON schema as input and generates C# classes and files as a model for the schema.

docs available at core.telegram.org are now pretty much up to date. (Layer 105)

Shouldn't we update TLSharp to layer 105 first?

And I don't know exactly if TeleSharp.Generator is enough for updating TLSharp to support higher layers or not. I just tried to correct the thing about the "Latest scheme".

knocte commented 4 years ago

I'm not really deep into TeleSharp.Generator but as far I know about this project, it gets JSON schema as input and generates C# classes and files as a model for the schema.

I believe so too. So my point is we should regenerate with Layer105 before committing this PR.

knocte commented 4 years ago

We've upgraded the layer in TgSharp, please help us test it.