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 ?

14 Replies

Favna•4mo ago

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

vladdy•4mo ago

Yes Probably Idk

kyra•4mo ago

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

vladdy•4mo ago

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

kyra•4mo ago

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

vladdy•4mo ago

rawName?

kyra•4mo ago

kebabCaseName, transformedName

vladdy•4mo ago

...no

kyra•4mo ago

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

vladdy•4mo ago

which is why i said lets make a rawName property

Favna•4mo ago

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

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

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

Calypso•3mo ago

Thanks!

Gaming

Programming

Registering Context Commands