grammyjs / website

The grammY documentation website.
https://grammy.dev
MIT License
46 stars 108 forks source link

Explicitly mention that lazy sessions do not support multi sessions #936

Open euberdeveloper opened 1 year ago

euberdeveloper commented 1 year ago

While we do mention that lazy sessions are a separate implementation of sessions, it can seem like there would be feature parity between the two plugins. We should make it explicit that this is not the case, and use multi sessions as the main example for things that lazy sessions do not support.

The original issue description is below this line.


ORIGINAL COMMENT

          Hang on, is this only breaking when you're using lazy multi sessions? If so, then the problem is that there are no lazy multi sessions. This isn't supported. We implemented it initially, but combining lazy sessions with multi sessions leads to numerous strange edge cases that are hard to understand and by design impossible to mitigate, so we dropped this again. If you were using TypeScript, your editor could've told you this. Since you don't seem to be using it, here is a link to the API reference of grammY which tells you this: https://deno.land/x/grammy@v1.12.4/mod.ts?s=lazySession

I'll add another example for this to the docs in the coming days. There is no bug here.

I will close this issue. Feel free to comment or reopen if you have further questions :)

Originally posted by @KnorpelSenf in https://github.com/grammyjs/grammY/issues/345#issuecomment-1366223793

PROBLEM

About the cited comment, it would be nice to have it written in the documentation here and maybe also a possible workaround example

KnorpelSenf commented 1 year ago

There are no lazy multi sessions. There is no way to document them. There also is no workaround. Lazy sessions and multi sessions are mutually exclusive.

euberdeveloper commented 1 year ago

Yes, I was pointing out that in the documentation you could have written exactly that.