Skip to content

fix: add missing update types to constants - #951

Open
KnightNiwrem wants to merge 1 commit into
grammyjs:v2from
KnightNiwrem:fix/v2-update-type-constants
Open

KnightNiwrem wants to merge 1 commit into
grammyjs:v2from
KnightNiwrem:fix/v2-update-type-constants

Conversation

@KnightNiwrem

Copy link
Copy Markdown
Member

Note

This pull request was primarily written by GPT-5.6 Sol xhigh in ChatGPT Work.

Summary

Adds the three supported update types that were omitted from both v2 update-type constants:

  • guest_message
  • managed_bot
  • subscription

The values are added to both DEFAULT_UPDATE_TYPES and ALL_UPDATE_TYPES. The latter currently duplicates the default list because of the existing JSR workaround.

Main-branch reference

The values and their ordering match DEFAULT_UPDATE_TYPES on main. On main, ALL_UPDATE_TYPES includes that list by spreading DEFAULT_UPDATE_TYPES.

Verification

  • deno fmt --check passes.
  • deno lint passes.
  • deno check src/convenience/constants.ts passes.
  • A runtime assertion confirms that all three values occur in both exported arrays.

The complete test task could not resolve @grammyjs/debug from JSR in the local environment; CI can run the full suite.

@KnightNiwrem
KnightNiwrem marked this pull request as ready for review August 15, 2026 17:57
@KnorpelSenf KnorpelSenf added the v2 related to grammY 2.0 label Aug 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v2 related to grammY 2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants