kyle13 Comments - Answer Overflow

kyle13

•Created by kyle13 on 4/5/2023 in #sapphire-support

Handling deletion of all guild specific slash commands with BulkOverwrite

Thank you again for correcting me. I only meant to say that I notice that commands can be removed by Sapphire which was due my misunderstanding of your statement being specifically about deletion vs overriding which seems can have the result of removing a command without explicitly calling delete. It's not my intention to argue anything, just trying to clarify my own understanding and I really appreciate the time you take answer questions and help me and others better utilize the Sapphire framework.

8 replies

SIASapphire - Imagine a framework

•Created by kyle13 on 4/5/2023 in #sapphire-support

Handling deletion of all guild specific slash commands with BulkOverwrite

Thanks @Favna , I don't think it's quite accurate to say Sapphire doesnt handle deletion of commands since it does delete both global application and guild specific commands when using BulkOverwrite which I've seen from my usage and this is also stated here. https://github.com/sapphiredev/framework/blob/465b05c/src/lib/types/Enums.ts#L55 The part I'm trying to solve is when an update makes it so a guild is no longer registered via guildIds, which as it says here is not covered by BulkOverwrite. the same applies for guild commands, but only for guilds that are registered in the registry via guildIds I'll look into using discordjs as you recommend though. Thanks for the update.

8 replies

SIASapphire - Imagine a framework

•Created by kyle13 on 3/21/2023 in #sapphire-support

Handling BulkOverwrite Registry Errors

Thank you @Vladdy and @Favna for your outstanding support! I was able to download the test version and verify adding a listener lets me capture the bulk overwrite errors perfectly. I'll go ahead and mark this request as solved.

@ApplyOptions<Listener.Options>({
  event: Events.ApplicationCommandRegistriesBulkOverwriteError,
})
export class ApplicationRegistryError extends Listener<
  typeof Events.ApplicationCommandRegistriesBulkOverwriteError
> {
  public run(error: unknown, guildId: string) {
    logger.error('caught bulk registry error', { guildId, error });
  }
}

@ApplyOptions<Listener.Options>({
  event: Events.ApplicationCommandRegistriesBulkOverwriteError,
})
export class ApplicationRegistryError extends Listener<
  typeof Events.ApplicationCommandRegistriesBulkOverwriteError
> {
  public run(error: unknown, guildId: string) {
    logger.error('caught bulk registry error', { guildId, error });
  }
}

37 replies

SIASapphire - Imagine a framework

•Created by kyle13 on 3/21/2023 in #sapphire-support

Handling BulkOverwrite Registry Errors

@A Vladdy the code fix looks like it will solve the issue. I'm just not sure how to "install the packed version" to test it out. I tried checking out the branch, building it and using npm link, but it seems like it can't resolve the types for my project when used that way. Any advice on how to install and test out your fix? Also is there a timeline on when this would be added to a release? Thanks so much for your help!

37 replies

SIASapphire - Imagine a framework

•Created by kyle13 on 3/21/2023 in #sapphire-support

Handling BulkOverwrite Registry Errors

Thanks @Vladdy appreciate the response and fast turnaround!

37 replies

Gaming

Programming