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•5mo ago

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

vladdy•5mo ago

Yes Probably Idk

kyra•5mo ago

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

vladdy•5mo ago

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

kyra•5mo ago

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

vladdy•5mo ago

rawName?

kyra•5mo ago

kebabCaseName, transformedName

vladdy•5mo ago

...no

kyra•5mo ago

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

vladdy•5mo ago

which is why i said lets make a rawName property

Favna•5mo ago

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

Calypso•5mo 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•5mo ago

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

Calypso•5mo ago

Thanks!

Mathias•7d 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),
    );
  }

Gaming

Programming

Registering Context Commands