grammyjs / grammY

The Telegram Bot Framework.
https://grammy.dev
MIT License
2.37k stars 117 forks source link

Update frameworks.ts #663

Closed BilagoNet closed 6 days ago

BilagoNet commented 6 days ago

74 | let resolveResponse; 75 | return { 76 | update: request.json(), 77 | header: request.headers.get(SECRET_HEADER) || undefined, 78 | end: () => { 79 | resolveResponse(ok()); ^ TypeError: resolveResponse is not a function. (In 'resolveResponse(ok())', 'resolveResponse' is undefined)

KnorpelSenf commented 6 days ago

Can you explain this change and why it is necessary?

BilagoNet commented 6 days ago

Can you explain this change and why it is necessary?

bot-1 | TypeError: resolveResponse is not a function. (In 'resolveResponse(ok())', 'resolveResponse' is undefined) bot-1 | at end (/usr/src/node_modules/grammy/out/convenience/frameworks.js:79:13) bot-1 | at /usr/src/node_modules/grammy/out/convenience/webhook.js:52:55 bot-1 | 74 | let resolveResponse; bot-1 | 75 | return { bot-1 | 76 | update: request.json(), bot-1 | 77 | header: request.headers.get(SECRET_HEADER) || undefined, bot-1 | 78 | end: () => { bot-1 | 79 | resolveResponse(ok()); bot-1 | ^ bot-1 | TypeError: resolveResponse is not a function. (In 'resolveResponse(ok())', 'resolveResponse' is undefined)

this problem show me, fixed with handlerReturn :)

KnorpelSenf commented 6 days ago

@all-contributors add @BilagoNet for bug and code

allcontributors[bot] commented 6 days ago

@KnorpelSenf

I've put up a pull request to add @BilagoNet! :tada:

BilagoNet commented 6 days ago

thanks :)

KnorpelSenf commented 6 days ago

This was released as 1.31.3