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

feat: Unbanned event addition #185

Merged
merged 7 commits into from
Jan 6, 2025

Conversation

Undid-Iridium
Copy link

Description

Describe the changes
https://discord.com/channels/656673194693885975/1307441157483270207

What is the current behavior? (You can also link to an open issue here)
Doesn't call Unban event

What is the new behavior? (if this is a feature change)
It calls Unban event now

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Not exactly. Function for this behaves differently so modifying the target is not easily done without breaking change so, just focusing on whether the player should or should not be unbanned should be sufficient.

Other information:
Not tested


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

@louis1706 Clarify if this is what you meant

@VALERA771
Copy link

I don't think that this will work cuz you specify only 1 method to patch and it may just put 2 transpilers on same method

@Undid-Iridium
Copy link
Author

Check

@louis1706 louis1706 added the Requires Testing Things need to be verified by an Exiled Developer/Contributor label Nov 22, 2024
@louis1706 louis1706 changed the title Unbanned event addition feat: Unbanned event addition Dec 26, 2024
Copy link

@VALERA771 VALERA771 left a comment

Choose a reason for hiding this comment

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

All good if works fine

@louis1706 louis1706 merged commit 5139cfc into ExMod-Team:dev Jan 6, 2025
5 of 6 checks passed
dreamsvxyz pushed a commit to DaybreakLabs/Vanced that referenced this pull request Jan 6, 2025
* Option A

* Style cop happy

* Sigh stylecop

* For Yamato

* Done, not tested

* Stylecop

---------

Co-authored-by: Yamato <[email protected]>
VALERA771 added a commit that referenced this pull request Jan 10, 2025
* feat: ``Lift::Get(ElevatorGroup)`` (#372)

Lift::Get(ElevatorGroup)

* fix: PlacingBulletHoleEventArgs::Position (#377)

Fix

* fix: DryFiring fired constantly (#378)

Fix DryFiring fired constantly

* feat: Unbanned event addition (#185)

* Option A

* Style cop happy

* Sigh stylecop

* For Yamato

* Done, not tested

* Stylecop

---------

Co-authored-by: Yamato <[email protected]>

* feat: Adding CustomKeycard for CustomItems (#98)

* uwu

* itemtype check

* bruh bruh brrr ☠

* fix

* fix

* fix no.2

* some events

* ExMod Team copyright

---------

Co-authored-by: Yamato <[email protected]>
Co-authored-by: Yamato <[email protected]>

* feat: ShadowType (#382)

More features for the Light wrapper

* fix: remove var & fix FF for explosions

* feat: DroppingAmmo, DroppedAmmo and InteractingDoor (#371)

* fix: DoorType and RoomType for Checkpoint (#379)

Fix DoorType and RoomType for Checkpoint

* feat: CustomStats (#363)

* CustomStats

* replace Mathf.Clamp01 by Mathf.Clamp

* Remove CustomStaminaStat.cs

---------

Co-authored-by: Yamato <[email protected]>

* fix: NPC Spawn using wrong SpawnType (#375)

Update Npc.cs

* fix: Fix FF for explosions (#383)

* fix: nre fix

* fix: respawningteam event fix

* fix: fix explosion ff

* feat: LockerType extension (#384)

* feat: LockerType extension

* fix: fix of fix

* fix: init fix for play gun sound

* Revert "fix: init fix for play gun sound"

This reverts commit f4d5541.

* feat: Fix OnInternalSpawning & Added ChargingJailbird setter (by BlackSerperior6) (#368)

BlackSerperior6

Co-authored-by: BlackSerperior6 <[email protected]>

* feat: `ExplodingGrenadeEventArgs::ExplosionType` (#385)

* ExplosionType

* ExplodingGrenadeEventArgs::ExplosionType

---------

Co-authored-by: Yamato <[email protected]>
Co-authored-by: scp252arc <[email protected]>
Co-authored-by: X <[email protected]>
Co-authored-by: Yamato <[email protected]>
Co-authored-by: Trevlouw <[email protected]>
Co-authored-by: Misfiy <[email protected]>
Co-authored-by: Cosmos Zvezdochkin <[email protected]>
Co-authored-by: Rysik5318 <[email protected]>
Co-authored-by: BlackSerperior6 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Requires Testing Things need to be verified by an Exiled Developer/Contributor Transpiler
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants