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

Breaking: Renamed localEulerAnglesSensitivity #1474

Merged
merged 1 commit into from
Feb 2, 2020

Conversation

MrGadget1024
Copy link
Collaborator

This feature was only added 2 weeks ago, so I think the rename is safe without an obsolete.

@miwarnec
Copy link
Collaborator

miwarnec commented Feb 2, 2020

why rename it?
seems like rotation sensitivity tells people exactly what it is.
someone new to unity might not understand what euler sensitivity could be

@paulpach
Copy link
Contributor

paulpach commented Feb 2, 2020

This is a breaking change

@paulpach
Copy link
Contributor

paulpach commented Feb 2, 2020

why rename it?
seems like rotation sensitivity tells people exactly what it is.
someone new to unity might not understand what euler sensitivity could be

He is renaming it from localeulerangle to local rotation sensitivity, not the other way around

@MrGadget1024
Copy link
Collaborator Author

He is renaming it from localeulerangle to local rotation sensitivity, not the other way around

@vis2k What he said ^

@MrGadget1024 MrGadget1024 changed the title Renamed localEulerAnglesSensitivity Breaking: Renamed localEulerAnglesSensitivity Feb 2, 2020
@MrGadget1024 MrGadget1024 added enhancement New feature or request PassedTestRoutine labels Feb 2, 2020
@paulpach paulpach merged commit eee9692 into master Feb 2, 2020
@paulpach paulpach deleted the rename-localEulerAnglesSensitivity branch February 2, 2020 19:35
github-actions bot referenced this pull request in MirageNet/Mirage Feb 6, 2020
# [8.0.0](7.0.0-master...8.0.0-master) (2020-02-06)

### Bug Fixes

* call callback after update dictionary in host ([#1476](https://github.com/MirrorNG/MirrorNG/issues/1476)) ([1736bb0](1736bb0))
* port network discovery ([d6a1154](d6a1154))
* remove scriptableobject error Tests ([479b78b](479b78b))
* Telepathy reverted to older version to fix freezes when calling Client.Disconnect on some platforms like Windows 10 ([d0d77b6](d0d77b6))
* Telepathy updated to latest version. Threads are closed properly now. ([4007423](4007423))

* Renamed localEulerAnglesSensitivity (#1474) ([eee9692](eee9692)), closes [#1474](https://github.com/MirrorNG/MirrorNG/issues/1474)

### breaking

* Transports can now provide their Uri ([#1454](https://github.com/MirrorNG/MirrorNG/issues/1454)) ([b916064](b916064)), closes [#38](#38)

### Features

* Implemented NetworkReaderPool ([#1464](https://github.com/MirrorNG/MirrorNG/issues/1464)) ([9257112](9257112))
* LAN Network discovery ([#1453](https://github.com/MirrorNG/MirrorNG/issues/1453)) ([e75b45f](e75b45f)), closes [#38](#38)
* Mirror Icon for all components ([#1452](https://github.com/MirrorNG/MirrorNG/issues/1452)) ([a7efb13](a7efb13))
* supports scriptable objects ([4b8f819](4b8f819))

### BREAKING CHANGES

* localEulerAnglesSensitivity renamed to localRotationSensitivity
* Make the server uri method mandatory in transports

Co-authored-by: MrGadget <chris@clevertech.net>
github-actions bot referenced this pull request in MirageNet/Mirage Feb 6, 2020
BREAKING CHANGE:  localEulerAnglesSensitivity renamed to localRotationSensitivity
github-actions bot referenced this pull request in MirageNet/Mirage Feb 6, 2020
# [8.0.0](7.0.0-master...8.0.0-master) (2020-02-06)

### Bug Fixes

* call callback after update dictionary in host ([#1476](https://github.com/MirrorNG/MirrorNG/issues/1476)) ([1736bb0](1736bb0))
* port network discovery ([d6a1154](d6a1154))
* remove scriptableobject error Tests ([479b78b](479b78b))
* Telepathy reverted to older version to fix freezes when calling Client.Disconnect on some platforms like Windows 10 ([d0d77b6](d0d77b6))
* Telepathy updated to latest version. Threads are closed properly now. ([4007423](4007423))

* Renamed localEulerAnglesSensitivity (#1474) ([eee9692](eee9692)), closes [#1474](https://github.com/MirrorNG/MirrorNG/issues/1474)

### breaking

* Transports can now provide their Uri ([#1454](https://github.com/MirrorNG/MirrorNG/issues/1454)) ([b916064](b916064)), closes [#38](#38)

### Features

* Implemented NetworkReaderPool ([#1464](https://github.com/MirrorNG/MirrorNG/issues/1464)) ([9257112](9257112))
* LAN Network discovery ([#1453](https://github.com/MirrorNG/MirrorNG/issues/1453)) ([e75b45f](e75b45f)), closes [#38](#38)
* Mirror Icon for all components ([#1452](https://github.com/MirrorNG/MirrorNG/issues/1452)) ([a7efb13](a7efb13))
* supports scriptable objects ([4b8f819](4b8f819))

### BREAKING CHANGES

* localEulerAnglesSensitivity renamed to localRotationSensitivity
* Make the server uri method mandatory in transports

Co-authored-by: MrGadget <chris@clevertech.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants