gray vortex
Hello, I just started to continue on my bot. I just did a upgrade to the most recent version of discord.js and sapphire. But now, the bot started giving me after "boot" cringe errors telling me that he can't register the commands. Exactly: take a look on the screenshot, this **** ist too long!
sapphire-version is "next". I use CommonJS btw, that was a previous error 😅
wary pulsarBOT
gray vortex
BTW, the code of one of the commands is: ```ts
import { PrismaClient } from '@prisma/client';
import { ApplyOptions } from '@sapphire/decorators';
import { Command } from '@sapphire/framework';
const prisma = new PrismaClient();
@ApplyOptions<Command.Options>({
description: 'Some description.'
})
export class UserCommand extends Command {
public constructor(context: Command.Context) {
super(context, {
preconditions: ['admin', 'guildchannel'],
cooldownDelay: 10_000
});
}
public override registerApplicationCommands(registry: Command.Registry) {
registry.registerChatInputCommand({
name: this.name,
description: this.description
});
}
public async chatInputRun(interaction: Command.ChatInputCommandInteraction) { /* some code */}}} // I didn't count the brackets```
jaunty steeple
its most likely to do with the fact that your providing the decorator and constructor. you have to provide one or the other, so you can either:
pure flax
Can you show the code for losungschannel command
gray vortex
its most likely to do with the fact that your providing the decorator and constr...
Okay, I'll try that as soon I'm home
gray vortex
BTW, the code of one of the commands is: ```ts
import { PrismaClient } from '@pr...
@pure flax that is the code.
gray vortex
its most likely to do with the fact that your providing the decorator and constr...
Is that a new feature of the latest update?
Because it worked until I did a upgrade to [email protected] 😅 (and the latest sapphire version of course 😉)
jaunty steeple
Is that a new feature of the latest update?
it's not new, no. but u should never mix class decorators and constructors because your basically including two constructors
gray vortex
Ok. I think I crossed two templates, I'll change this.
gray vortex
I just removed the decorators from every command having a constructor. Sapphire stopped showing this error, and loads the commands properly, so I think this issue is done for now. Thanks @jaunty steeple !!!
wary pulsarBOT
pure flax
well you should remove the contructor instead
jaunty steeple
I just removed the decorators from every command having a constructor. Sapphire ...
you can still use decorators but you would just have to remove the constructor. for example: https://github.com/sapphiredev/utilities/blob/main/packages/decorators/src/piece-decorators.ts#L15-L27
maybe this will help visualize what i mean, whilst trying not to spoonfeed
public constructor(context: <Class>.Context) {
super(context, {
// you would move everything in this object, into the decorators object
// after moving everything, remove the whole of this constructor method
// leaving you with your decorator, registerApplicationCommands, chatInputRun and so on
});
}