Skip to content

fix: PlayerReceivingLoadoutEventArgs from LabAPI#773

Open
TiBarification wants to merge 4 commits intoExMod-Team:devfrom
TiBarification:fix/labapi-loadout
Open

fix: PlayerReceivingLoadoutEventArgs from LabAPI#773
TiBarification wants to merge 4 commits intoExMod-Team:devfrom
TiBarification:fix/labapi-loadout

Conversation

@TiBarification
Copy link

Description

Re-adds logic of invoking events for loadouts

What is the current behavior? (You can also link to an open issue here)
LabApi events of loadout broken

What is the new behavior? (if this is a feature change)
LabApi events of loadout triggered

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
no

Other information:


Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentations

Submission checklist

  • I have checked the project can be compiled
  • I have tested my changes and it worked as expected

Patches (if there are any changes related to Harmony patches)

  • I have checked no IL patching errors in the console

Other

  • Still requires more testing

Copilot AI review requested due to automatic review settings March 8, 2026 23:38
@TiBarification TiBarification changed the base branch from master to dev March 8, 2026 23:39
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Re-introduces LabApi loadout event invocation during role changes so external LabApi plugins can observe/cancel loadout application again.

Changes:

  • Adds LabApi ReceivingLoadout event invocation (with allow/deny gate) before inventory modifications.
  • Adds LabApi ReceivedLoadout event invocation after items are granted.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@louis1706 louis1706 left a comment

Choose a reason for hiding this comment

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

This will make it fully usable

@louis1706 louis1706 changed the title fix: trigger labapi loadout fix: PlayerReceivingLoadoutEventArgs from LabAPI Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants