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.

Sign up for GitHub

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

Jump to bottom

Ignore NPCs in JailListener #5130

Merged
merged 6 commits into from
Oct 20, 2022
Merged

Ignore NPCs in JailListener #5130

merged 6 commits into from
Oct 20, 2022

Conversation

JRoy
Copy link
Member

@JRoy JRoy commented Oct 11, 2022

these aren't needed, npcs aren't ever jailed

@JRoy JRoy added type: enhancement Features and feature requests. module: main Issues or PRs for the main Essentials module labels Oct 11, 2022
@JRoy JRoy added this to the 2.20.0 milestone Oct 11, 2022
@JRoy JRoy requested a review from pop4959 October 15, 2022 00:44
@JRoy JRoy requested a review from pop4959 October 18, 2022 14:41
@mdcfe mdcfe enabled auto-merge (squash) October 20, 2022 13:36
@mdcfe mdcfe changed the title Ignore NPCs with JailListener Ignore NPCs in JailListener Oct 20, 2022
@mdcfe mdcfe disabled auto-merge October 20, 2022 13:37
@mdcfe mdcfe enabled auto-merge (squash) October 20, 2022 13:37
@mdcfe mdcfe merged commit 879e70f into EssentialsX:2.x Oct 20, 2022
@JRoy JRoy deleted the fix/dont-call-jail-npc branch October 20, 2022 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mdcfe mdcfe mdcfe approved these changes

@pop4959 pop4959 Awaiting requested review from pop4959

Assignees
No one assigned
Labels
module: main Issues or PRs for the main Essentials module type: enhancement Features and feature requests.
Projects
None yet
Milestone
2.20.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@JRoy @mdcfe @pop4959