-
Notifications
You must be signed in to change notification settings - Fork 102
feat: New Player Changed Room events #557
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
Conversation
|
I am not sure than ZoneChanged is actually needed since RoomChanged will be call anyways and a simple check would do the work |
@louis1706 hmm yes, I'm undecided about this, I leave the choice to you, if you want, you can delete the zone changed event, or I can delete it, just say |
Yeah in my opinion unneeded event need to be removed it's reduced useless complexity inside transpiller |
|
@louis1706 Deleted Zone Changed Event |
Description
Describe the changes
Added RoomChanged and ZoneChanged events triggered when a player changes rooms or zones.
What is the current behavior? (You can also link to an open issue here)
The Exiled does not provide built-in events for tracking room/zone changes.
Plugins must manually track player Rooms or Zones to detect these changes.
What is the new behavior? (if this is a feature change)
Plugins can now subscribe to:
Player.RoomChanged: Fires when a player enters a new RoomIdentifier.
Player.ZoneChanged: Fires when a player enters a new ZoneType.
Both events include Player and old room , new room or old zone new zone data(zones just for changed zone event ) .
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
❌ No. This is a non-breaking change that adds new features without modifying existing behavior.
Other information:
Types of changes
Submission checklist
Patches (if there are any changes related to Harmony patches)
Other