Skip to content

Conversation

@Cleboost
Copy link
Owner

  • Implémentation de la classe ContextMenu pour gérer les interactions de menu contextuel.
  • Ajout de la gestion des interactions de menu contextuel dans le client.
  • Mise à jour des gestionnaires pour inclure le nouveau type d'interaction.
  • Ajout d'un exemple de menu contextuel dans le playground.

- Implémentation de la classe ContextMenu pour gérer les interactions de menu contextuel.
- Ajout de la gestion des interactions de menu contextuel dans le client.
- Mise à jour des gestionnaires pour inclure le nouveau type d'interaction.
- Ajout d'un exemple de menu contextuel dans le playground.
@Cleboost Cleboost requested a review from Copilot April 21, 2025 12:11
@Cleboost Cleboost linked an issue Apr 21, 2025 that may be closed by this pull request
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces support for context menus by adding a new ContextMenu class, its corresponding handler, and integrating it into the loader and event systems.

  • Added a ContextMenu example in the playground.
  • Integrated ContextMenu in the handler loader, events, and API push.
  • Introduced a new ContextMenu handler in both djs-core and djs-cli.

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
playground/src/interactions/context/infos.ts Added a context menu example with a minor typo in the reply message.
packages/djs-core/src/index.ts Exposed the ContextMenu class via the package index.
packages/djs-core/src/handlers/loader.ts Updated the handler loader to register context menus.
packages/djs-core/src/handlers/events.ts Updated event routing to handle context menu interactions.
packages/djs-core/src/handlers/ContextMenu.ts Introduced a new context menu handler (note a spelling inconsistency in the class name).
packages/djs-core/src/class/interactions/ContextMenu.ts Added the ContextMenu class implementation.
packages/djs-core/src/class/BotClient.ts Integrated the context menu handler into the BotClient.
packages/djs-cli/src/index.ts Updated hot reload to support context menu interactions.
Comments suppressed due to low confidence (2)

packages/djs-core/src/handlers/ContextMenu.ts:17

  • The class name 'ContetxtMenuHandler' appears misspelled. Consider renaming it to 'ContextMenuHandler' for consistency and clarity.
export default class ContetxtMenuHandler {

packages/djs-core/src/class/BotClient.ts:26

  • The imported handler name 'ContetxtMenuHandler' is misspelled. It should reflect 'ContextMenuHandler' to match standard naming conventions.
import ContetxtMenuHandler from "../handlers/ContextMenu";

@Cleboost Cleboost merged commit 54a372d into master Apr 21, 2025
4 checks passed
@Cleboost Cleboost deleted the 77-feature-contextmenu-interaction branch July 2, 2025 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE]: ContextMenu Interaction

2 participants