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

[FR:] /plot setowner(s) | /plot addowner(s) & /plot removeowner(s) #45

Open
supercrafter333 opened this issue Jul 31, 2022 · 1 comment
Labels
Category: UI Related to the user interface (e.g. commands, terminal output) Status: For discussion Still in need of further discussion regarding the topic Type: Enhancement
Milestone

Comments

@supercrafter333
Copy link
Contributor

I saw that CPlot haven't implemented commands like /plot setowner(s) | /plot addowner(s) & /plot removeowner(s) yet.
Plot-Owners cannot add or remove owners at this time, but I think the most CPlot users would be very happy about these commands.
I think it would be very usefull to add these commands.
These commands should also merge the owners from merged plots.

@ColinHDev ColinHDev added Type: Enhancement Category: UI Related to the user interface (e.g. commands, terminal output) Status: For discussion Still in need of further discussion regarding the topic labels Jul 31, 2022
@ColinHDev
Copy link
Owner

I think, I will address all of that in a single PR, which will probably include:

  • /p setowner as an admin command, to set the owner of a plot
  • Maybe /p addowner for plot owners to add other players as owners
  • Not /p removeowner: Being the plot owner obviously means you own the plot. So there won't be a feature for plot owners to remove other plot owners. If someone is an owner, he will stay that unless an admin decides to change that.
  • /p give, although I am not sure what to do if a plot has multiple owners. Maybe those others need to accept with /p give ColinHDev confirm
  • Change /p merge, so admins can merge plots with different owners, with both of them becoming owners of the plot. Maybe, also allowing normal players to merge plots that they do not own and only merge if the other's plot owners accept with p merge ColinHDev confirm

@ColinHDev ColinHDev mentioned this issue Aug 1, 2022
@ColinHDev ColinHDev added this to the v1.0.0 milestone Aug 1, 2022
@ColinHDev ColinHDev mentioned this issue Aug 17, 2022
Closed
@ColinHDev ColinHDev mentioned this issue Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: UI Related to the user interface (e.g. commands, terminal output) Status: For discussion Still in need of further discussion regarding the topic Type: Enhancement
Projects
None yet
Development

No branches or pull requests

2 participants