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: adding RemoveListener and RemoveAllListeners to AddLateEvent #764

Merged
merged 1 commit into from
Apr 7, 2021

Conversation

James-Frowen
Copy link
Member

BREAKING CHANGE: AddLateEvent Reset no longer removes listeners

BREAKING CHANGE: AddLateEvent Reset no longer removes listeners
@sonarcloud
Copy link

sonarcloud bot commented Apr 7, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

85.7% 85.7% Coverage
0.0% 0.0% Duplication

@Lymdun Lymdun merged commit d67c96c into master Apr 7, 2021
@James-Frowen James-Frowen deleted the addLateEvent2 branch April 8, 2021 15:53
github-actions bot pushed a commit that referenced this pull request Apr 8, 2021
# [87.0.0](v86.0.2...v87.0.0) (2021-04-08)

### Bug Fixes

* using add late event for authority and combining start and stop events ([#767](#767)) ([8903f00](8903f00))
* using AddLateEvent to stop race condition for client events  ([#768](#768)) ([681875b](681875b))
* using AddLateEvent to stop race condition for network identity events ([#766](#766)) ([4f8bf11](4f8bf11))
* using AddLateEvent to stop race condition for server events ([#765](#765)) ([a1ec84c](a1ec84c))

### Features

* adding RemoveListener and RemoveAllListeners to AddLateEvent ([#764](#764)) ([d67c96c](d67c96c))
* adding RemoveListener and RemoveAllListeners to AddLateEvent ([#764](#764)) ([0ca2804](0ca2804))

### BREAKING CHANGES

* - NetworkClient.Connected event is now type of IAddLateEvent
- NetworkClient.Authenticated event is now type of IAddLateEvent
- NetworkClient.Disconnected event is now type of IAddLateEvent

* refactor: removing NetworkConnectionEvent use NetworkPlayerEvent instead
* NetworkConnectionEvent renamed to NetworkPlayerEvent

* reverting clean up change
* - Identity.OnStartAuthority and IdentityOnStopAuthority are now Identity.OnAuthorityChanged and are type of IAddLateEvent<bool>
* - Server.Started event is now type of IAddLateEvent
- Server.Stoped event is now type of IAddLateEvent
- Server.OnStartHost event is now type of IAddLateEvent
- Server.OnStopHost event is now type of IAddLateEvent
- inspector values for changed events will need to be re-assigned
* - Identity.OnStartServer event is now type of IAddLateEvent
- Identity.OnStopServer event is now type of IAddLateEvent
- Identity.OnStartClient event is now type of IAddLateEvent
- Identity.OnStopClient event is now type of IAddLateEvent
- Identity.OnStartLocalPlayer event is now type of IAddLateEvent
- inspector values for changed events will need to be re-assigned
* AddLateEvent Reset no longer removes listeners
* AddLateEvent Reset no longer removes listeners
@github-actions
Copy link
Contributor

github-actions bot commented Apr 8, 2021

🎉 This PR is included in version 87.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

dragonslaya84 pushed a commit that referenced this pull request Apr 11, 2021
)

BREAKING CHANGE: AddLateEvent Reset no longer removes listeners
dragonslaya84 pushed a commit that referenced this pull request Apr 11, 2021
)

BREAKING CHANGE: AddLateEvent Reset no longer removes listeners
dragonslaya84 pushed a commit that referenced this pull request Apr 11, 2021
# [87.0.0](v86.0.2...v87.0.0) (2021-04-08)

### Bug Fixes

* using add late event for authority and combining start and stop events ([#767](#767)) ([8903f00](8903f00))
* using AddLateEvent to stop race condition for client events  ([#768](#768)) ([681875b](681875b))
* using AddLateEvent to stop race condition for network identity events ([#766](#766)) ([4f8bf11](4f8bf11))
* using AddLateEvent to stop race condition for server events ([#765](#765)) ([a1ec84c](a1ec84c))

### Features

* adding RemoveListener and RemoveAllListeners to AddLateEvent ([#764](#764)) ([d67c96c](d67c96c))
* adding RemoveListener and RemoveAllListeners to AddLateEvent ([#764](#764)) ([0ca2804](0ca2804))

### BREAKING CHANGES

* - NetworkClient.Connected event is now type of IAddLateEvent
- NetworkClient.Authenticated event is now type of IAddLateEvent
- NetworkClient.Disconnected event is now type of IAddLateEvent

* refactor: removing NetworkConnectionEvent use NetworkPlayerEvent instead
* NetworkConnectionEvent renamed to NetworkPlayerEvent

* reverting clean up change
* - Identity.OnStartAuthority and IdentityOnStopAuthority are now Identity.OnAuthorityChanged and are type of IAddLateEvent<bool>
* - Server.Started event is now type of IAddLateEvent
- Server.Stoped event is now type of IAddLateEvent
- Server.OnStartHost event is now type of IAddLateEvent
- Server.OnStopHost event is now type of IAddLateEvent
- inspector values for changed events will need to be re-assigned
* - Identity.OnStartServer event is now type of IAddLateEvent
- Identity.OnStopServer event is now type of IAddLateEvent
- Identity.OnStartClient event is now type of IAddLateEvent
- Identity.OnStopClient event is now type of IAddLateEvent
- Identity.OnStartLocalPlayer event is now type of IAddLateEvent
- inspector values for changed events will need to be re-assigned
* AddLateEvent Reset no longer removes listeners
* AddLateEvent Reset no longer removes listeners
James-Frowen added a commit to James-Frowen/Mirage that referenced this pull request Apr 15, 2021
…irageNet#764)

BREAKING CHANGE: AddLateEvent Reset no longer removes listeners
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants