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

fix: PlayerEnterPlotEvent (java doc) description #4426

Merged

Conversation

RedstoneFuture
Copy link
Member

Overview

In this small Pull-Request the description of the PlayerEnterPlotEvent is fixed.

Description

  • The event should be triggered when the player "enters" the plot. And not when he "leaves" it.
  • The wording now corresponds to the description of the PlayerLeavePlotEvent event.

Submitter Checklist

Edit tasklist title
Beta Give feedback Tasklist Submitter Checklist, more options

Delete tasklist

Delete tasklist block?
Are you sure? All relationships in this tasklist will be removed.
  1. Make sure you are opening from a topic branch (/feature/fix/docs/ branch (right side)) and not your main branch.
    Options
  2. Ensure that the pull request title represents the desired changelog entry.
    Options
  3. New public fields and methods are annotated with @since TODO.
    Options
  4. I read and followed the contribution guidelines.
    Options

@RedstoneFuture RedstoneFuture requested a review from a team as a code owner May 12, 2024 20:25
@github-actions github-actions bot added the Bugfix This PR fixes a bug label May 12, 2024
@RedstoneFuture RedstoneFuture changed the title Fixing event description fix: PlayerEnterPlotEvent (java doc) description May 12, 2024
@dordsor21 dordsor21 merged commit c65c9e7 into IntellectualSites:main May 13, 2024
7 checks passed
@RedstoneFuture RedstoneFuture deleted the fix/event-description branch May 13, 2024 19:25
pull bot pushed a commit to Craftstuebchen/PlotSquared that referenced this pull request May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bugfix This PR fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants