Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Give his deadchest to another player #4

Open
apavarino opened this issue May 2, 2021 · 3 comments
Open

Give his deadchest to another player #4

apavarino opened this issue May 2, 2021 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@apavarino
Copy link
Owner

Problem :
Sometime player want to give his deadchest to another player.

Solution :
Give own deadchest to someone else by command (with a new permission node)

@apavarino apavarino added the enhancement New feature or request label May 2, 2021
@apavarino apavarino self-assigned this May 2, 2021
@ldemoulins
Copy link
Contributor

Shall it be considered that their might be different options on the command? I can see at least 2 cases that players might use:

  1. I died, and I want another specific player to get my deadchest that one time only
  2. I have a friend I trust on a server, and I want them to be able to get my deadchest everytime I die

For a bit more context, let's assume that the player that died is Crylonz and the player that might open the chest is Pandeul

For (1) -> Intuitively it seems like Pandeul should have access only to the last deadchest of Crylonz that was created at the time the command was issued. Meaning that if Crylonz dies between the time he issues the command and the time Pandeul picks up the chest, Pandeul can still open the chest Crylonz authorised him to, without having him be able to open any other chests on the map.
Alternatively, it could also be that Pandeul has the right to open ANY of Crylonz's chests, but only once.

For (2) -> Also need to have the reverse command, shouldn't it? Like if Crylonz sets Pandeul as having the right to get his deadchests but it was a mistake, he might want to remove Pandeul from the list.

Hop this helps the thought process :)

@CraftersX
Copy link

When to expect this add-on?

@apavarino
Copy link
Owner Author

I hope soon !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants