Registering Context Commands

Hello I want to register a context command with this code :

export class ValidRoroCommand extends Command {
  constructor(context, options) {
    super(context, {
      ...options,
      name: "Valide",
    });
  }

  registerApplicationCommands(registry) {
    registry.registerContextMenuCommand(builder =>
      builder //
        .setName(this.name)
        .setType(ApplicationCommandType.Message),
    );
  }

export class ValidRoroCommand extends Command {
  constructor(context, options) {
    super(context, {
      ...options,
      name: "Valide",
    });
  }

  registerApplicationCommands(registry) {
    registry.registerContextMenuCommand(builder =>
      builder //
        .setName(this.name)
        .setType(ApplicationCommandType.Message),
    );
  }

It's work but I don't have capital letter on the first letter V. Why ?

19 Replies

Favna•11mo ago

@vladdy @kyra 🩵🩷🤍🩷🩵 do we toLowerCase names when adding them to the command store :kekw:?

vladdy•11mo ago

Yes Probably Idk

kyra•11mo ago

Convert it to kebab-case in any case, probably?

vladdy•11mo ago

Hmm, definitely not in pieces Maybe in commands? Yep, we lowercase commands And changing that would be breaking

kyra•11mo ago

Or we can make it non-breaking by making it a getter we can use for the registry

vladdy•11mo ago

rawName?

kyra•11mo ago

kebabCaseName, transformedName

vladdy•11mo ago

...no

kyra•11mo ago

Vlad, changing name would change the registered name in the store, which will break some bots

vladdy•11mo ago

which is why i said lets make a rawName property

Favna•11mo ago

@Calypso can you create a GH issue to request a rawName property? The property should receive the name completely unmodified.

CalypsoOP•11mo ago

Good => https://github.com/sapphiredev/framework/issues/747

GitHub

request: rawName property for context commands · Issue #747 · sapph...

Is there an existing issue or pull request for this? I have searched the existing issues and pull requests Feature description The problem is that the command name is set to lowercase : export clas...

Favna•11mo ago

looking into this now @Calypso released 5.2.0, see #Announcements

CalypsoOP•11mo ago

Thanks!

Mathias•7mo ago

I can't get it to work! Do I need to do anything specific? My code:

export class ReportMessageCommand extends Command {
  constructor(context, options) {
    super(context, {
      ...options,
      name: "Report Message",
    });
  }

  registerApplicationCommands(registry) {
    registry.registerContextMenuCommand(builder =>
      builder //
        .setName(this.name)
        .setType(ApplicationCommandType.Message),
    );
  }

export class ReportMessageCommand extends Command {
  constructor(context, options) {
    super(context, {
      ...options,
      name: "Report Message",
    });
  }

  registerApplicationCommands(registry) {
    registry.registerContextMenuCommand(builder =>
      builder //
        .setName(this.name)
        .setType(ApplicationCommandType.Message),
    );
  }

vladdy•7mo ago

This is very vague, whats not working?

Mathias•7mo ago

My command name in the menu is still displayed in lowercase report message instead Report Message

vladdy•7mo ago

use this.rawName in the setName call

Mathias•7mo ago

Oh ok thanks ... I hadn't understood it like that in the code

Gaming

Programming

Registering Context Commands

Did you find this page helpful?