EssentialsX

EssentialsX

2M Downloads

[Feature request] Confirm for doing /clear

Sikatsu opened this issue ยท 11 comments

commented

Often players make a mistake with /clear causing their inventory to be reset.
Theres a confirm feature for /pay, but why not for /clear?

WARNING: This will clear your inventory, to confirm type /clear again

commented

@SupaHam Are you able to implement this perhaps?

commented

No promises. I'm quite busy.

commented

Thoughts on #1623?

commented

@SupaHam Looks nice. Hope you will merge it soon.

commented

@SupaHam

The new toggle command seems to give an error: https://pastebin.com/E8kvM0GX
But the confirm works fine.

commented

The class Commandclearconfirmtoggle should be Commandclearinventoryconfirmtoggle instead.

commented

Should be fixed now. Sorry about that.

commented

@SupaHam Builds are failing..

commented

This has been fixed - grab the latest build from the build server.

commented

I feel like if we keep adding confirmation prompts, we might as well make them generic and configurable - for example having a list in the config that contains all the EssentialsX commands that will require confirmation.

commented

That would be a nice idea too. I think however a confirmation for /ci is quite important to have.