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

[Exiled::Events] Adding AddingObserver and RemovingObserver events for Scp173 #2384

Merged
merged 6 commits into from
Feb 3, 2024

Conversation

skyfr0676
Copy link
Contributor

@skyfr0676 skyfr0676 commented Jan 13, 2024

the adding observer is triggered when a user just saw scp 173. for example, if ev.IsAllowed is set to false and the user is the only one who can saw scp173, the peanut can keep moving (it will be not added in the Observers list)

an example when the ev.IsAllowed is set to false
image

the RemoveObserver is the same as the AddingObserver event but there is no a IsAllowed parameter (it's weird in this situation) and it's trigger when the player no longer seen scp173.

special thanks to Bolton who help me to find stupid IL errors
special thing to yamato: i have compile this version in release build :trollface:

edit: ho i forgot to tell: i have build all change and tested the events with a different plugin in my personnal server

…ain)

the adding observer is triggered when a user just saw scp 173. for example, if ev.IsAllowed is set to false and the user is the only one who can saw scp173, the peanut can keep moving (it will be not added in the Observers list)

an example when the ev.IsAllowed is set to false
![image](https://github.com/Exiled-Team/EXILED/assets/99112969/35454fc1-938a-4c89-bc63-deb0d4dbfbf1)

he RemovingObserver is the same as the AddingObserver event but there is no a IsAllowed parameter (it's weird in this situation) and it's trigger when the player no longer saw scp173.

special thanks to Bolton who help me to find stupid IL errors
special thing to yamato: i have compile this xmas-2023 version in release build :trollface:

edit: ho i forgot to tell: i have build all change and tested the events with a different plugin in my personnal server
@skyfr0676 skyfr0676 changed the base branch from master to dev January 13, 2024 10:07
@skyfr0676
Copy link
Contributor Author

i think it's good now

@louis1706 louis1706 added this to the Exiled 9 milestone Jan 13, 2024
@NaoUnderscore NaoUnderscore changed the base branch from dev to apis-rework January 13, 2024 18:18
Copy link
Collaborator

@NaoUnderscore NaoUnderscore left a comment

Choose a reason for hiding this comment

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

Maintain all <param name=>, and such things, on the same line.

Exiled.Events/Patches/Events/Scp173/Observers.cs Outdated Show resolved Hide resolved
Exiled.Events/EventArgs/Scp173/RemoveObserverEventArgs.cs Outdated Show resolved Hide resolved
Exiled.Events/EventArgs/Scp173/RemoveObserverEventArgs.cs Outdated Show resolved Hide resolved
Exiled.Events/EventArgs/Scp173/AddingObserverEventArgs.cs Outdated Show resolved Hide resolved
Exiled.Events/EventArgs/Scp173/AddingObserverEventArgs.cs Outdated Show resolved Hide resolved
fixing nao change requested (comment lines)
@NaoUnderscore NaoUnderscore added enhancement New feature or request P2 Medium priority labels Jan 25, 2024
@NaoUnderscore NaoUnderscore changed the title adding AddingObserver event and RemovingObserver event for Scp173 (again) [EXiled::Events] Adding AddingObserver and RemovingObserver events for Scp173 Jan 29, 2024
NaoUnderscore and others added 2 commits January 29, 2024 15:32
- change `Exiled.API.Features.Pools` using to `Exiled.API.Features.Core.Generic.Pools`
@skyfr0676 skyfr0676 changed the title [EXiled::Events] Adding AddingObserver and RemovingObserver events for Scp173 [Exiled::Events] Adding AddingObserver and RemovingObserver events for Scp173 Jan 30, 2024
@NaoUnderscore
Copy link
Collaborator

Fix build errors

@NaoUnderscore NaoUnderscore merged commit f623959 into Exiled-Team:apis-rework Feb 3, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants