Skip to content

Conversation

@lucko
Copy link
Contributor

@lucko lucko commented Mar 21, 2021

Pull Request

Type

  • Internal change (Doesn't affect end-user).
  • External change (Does affect end-user).
  • Wiki (Changes towards the Wiki).
  • Other: __________

Description

Updates the documentation to correctly point to the LuckPerms wiki for a full list of placeholders.

I considered just updating the list to include the new/altered placeholders, however, I think:

  • it makes sense to have a single source - only one place to update in the future
  • the descriptions (missing from the docs here) are quite important for end-users to see and understand, as it is not immediately obvious what some of the placeholders do just by their name.

Thanks :)

@lucko lucko requested a review from Andre601 as a code owner March 21, 2021 13:56
Copy link
Collaborator

@Andre601 Andre601 left a comment

Choose a reason for hiding this comment

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

Thank you for this update

@Andre601 Andre601 merged commit a0b256c into PlaceholderAPI:docs/wiki Mar 21, 2021
@lucko lucko deleted the patch-1 branch March 21, 2021 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants