grammyjs / menu

Interactive menus for grammY.
https://grammy.dev/plugins/menu
MIT License
31 stars 11 forks source link

Redesign menu outdated detection #13

Closed KnorpelSenf closed 2 years ago

KnorpelSenf commented 3 years ago

It fires too many false positives. The timestamp example from the docs as well as the random number example both don't actually work the way they should.

One definite action to take is to remove payload comparison. It does not make sense that the payload needs to be regenerated at the time the handler is accessed. This defeats the entire point of storing data.

KnorpelSenf commented 3 years ago

Also, the fact that the button label is compared can be problematic in some cases—it is absolutely essential in most others.

Could we for example disregard all other inputs when the fingerprint is generated?