Slash command not updated
Hi, I have a move channel command with options setup as in picture. On the channel command I did not previously have the channel type option set. I have since updated it (some months ago) and it looks like the slash command was not updated since I can still use category type channels. On my dev bot I unregistered and reregistered the command and that fixed it. Is this a bug or?
24 Replies
hm yeah it looks like channel types arent considered in compute differences
@vladdy didn't you fix that? @Ravelt are you fully up-to-date?
I thought i did
Yes
I dont think you did @vladdy
H u h
there is nothing about channel types here: https://github.com/sapphiredev/framework/blob/134ca45f496de6a4928b670eaa8022eda8a56630/src/lib/utils/application-commands/compute-differences/option.ts
O shit
we mandela'd ourselves
Ok I'll run a sweeping check across all options then
Anyway that explains why it doesnt work @Ravelt. This will need fixing in the framework.
I think for now you can use bulk overwrite to circumvent the issue
Thank fuck i split this into multiple files btw
indeed that's why I was able to find it
Cool thanks
So how is bulk overwrite behaviour different from the default behaviour? It'll just update every command instead of just commands with detected differences? Is there any downsides to that? If no why isnt bulk overwrite the default behaviour?
Correct, no downsides, I think it should be the default but @vladdy doesn't and idk why either
methinks he doesn't doesn't want his baby to be shafted
Something about that it would undermine commands registered externally but that is for the whole 1 person that does that alongside sapphire I think
Changing the default now would mean releasing a breaking change for something that's...eh
reminder 👀
wait you're taking the break serious aren't you
Not intentionally
I am out with family, will be back monday morning
Is this why I keep having to use bulkoverwrite?
if you use channel types then yes
@vladdy reminder
bless
and the massive test file for compute differences continues to grow