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

Add argument to /skull to give skull to other player #5822

Merged
merged 6 commits into from
Nov 25, 2024

Conversation

Yurakaii
Copy link
Contributor

@Yurakaii Yurakaii commented Jun 5, 2024

Information

This PR closes #5762.

Details

Proposed feature:
This adds the ability to give a skull to another player using the /skull command with a second argument, if you only provide one argument the command will work as it previously did.

The new usage of the skull command is:
/<command> [owner] [player]

Environments tested:

OS: Windows 11

Java version: 21

  • Most recent Paper version (1.20.6, git-Paper-137)
  • CraftBukkit/Spigot/Paper 1.12.2
  • CraftBukkit 1.8.8

Demonstration:

This gives the dinnerbone's skull to trashdrumlinfly:
Screenshot 2024-06-05 144150

trashdrumlinfly received dinnerbone's skull
Screenshot 2024-06-05 143929

The command can still be used as how it worked before by just providing the first argument.

Copy link
Member

@JRoy JRoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

overall looks fine, can you just update the skullCommandUsage messages in the messages.properties file

@Yurakaii Yurakaii marked this pull request as draft July 23, 2024 21:14
@Yurakaii Yurakaii marked this pull request as ready for review July 23, 2024 21:30
@Yurakaii Yurakaii requested a review from JRoy July 23, 2024 21:32
@CatTeaA

This comment was marked as spam.

@Huge-mistake
Copy link

overall looks fine, can you just update the skullCommandUsage messages in the messages.properties file

Apologies for the interruption, can this PR be merged now? I would like to utilize this feature in the menu to provide players with skulls.

@JRoy JRoy added type: enhancement Features and feature requests. module: main Issues or PRs for the main Essentials module labels Nov 25, 2024
@JRoy JRoy changed the title Add give argument to skull command Add argument to /skull to give skull to other player Nov 25, 2024
@JRoy JRoy merged commit 38e42f9 into EssentialsX:2.x Nov 25, 2024
1 check passed
@JRoy
Copy link
Member

JRoy commented Nov 25, 2024

Thanks for the contribution :)

Euphillya added a commit to Euphillya/Essentials-Folia that referenced this pull request Dec 10, 2024
EssentialsX@1778bf5 Respect per player locale for command descriptions (EssentialsX#5972)
EssentialsX@2418a6f Improvements to Random Teleport (EssentialsX#4271)
EssentialsX@38e42f9 Add argument to /skull to give skull to other player (EssentialsX#5822)
EssentialsX@1af1565 Discord: Add start time placeholder for start message (EssentialsX#5686)
EssentialsX@bea43e8 Fix mini message parsing in NumberUtil#displayCurrency (EssentialsX#5921)
EssentialsX@3203e97 Parse /pay modifiers more strictly (EssentialsX#5638)
EssentialsX@f02eeec Add method to get all linked players in DiscordLinkService (EssentialsX#5896)
EssentialsX@9590d4c Add translation key for /alts output (EssentialsX#5595)
EssentialsX@3f0a412 Add locale based comma support for /pay (EssentialsX#5962)
EssentialsX@57c9edc Add optional requirements for balance top listing (EssentialsX#5394)
EssentialsX@c7cc1b4 Fix respawn-at-anchor setting. (EssentialsX#5825)
EssentialsX@2a41ea0 Add config options for specific chat type formats
EssentialsX@b560bbd Fix actions permissions on PRs for junit reports
EssentialsX@d31ff55 Remove extraneous code in command preprocess event handling
EssentialsX@bc8aa76 Revert removal of InvalidWorldException (EssentialsX#5984)
EssentialsX@d58db0d Update to 1.21.4
EssentialsX@c37064f Add 1.21.4 Mobs
EssentialsX@baa8258 Add 1.21.4 Tree Types
EssentialsX@424816e Add 1.21.4 Items
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: main Issues or PRs for the main Essentials module type: enhancement Features and feature requests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add an option to give someone their skull using /skull [player/skin] [player]
4 participants