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

Observer Reset Config Option #3

Merged
merged 3 commits into from Jul 11, 2013
Merged

Observer Reset Config Option #3

merged 3 commits into from Jul 11, 2013

Conversation

sanny-io
Copy link
Member

@sanny-io sanny-io commented Jul 7, 2013

Added config option for whether or not the player's position should be reset upon exiting observer mode. By default, it is set to true.

duck added 3 commits July 7, 2013 14:31
Added a config option for resetting the player's position after exiting observer mode.
Include new cl_plugin for adding the config option.
ObserverReset config option added.
@kurozael
Copy link
Member

Can you rename observerReset to observer_reset to keep with the naming convention used for config variables and then re-commit? Also this code:

if(observerReset == true) then

You're not putting a space before the ( which is part of our coding standard. Also if you could just do if (observerReset) instead of == true, as (observerReset) evaluates to true anyway.

Thanks!

@kurozael kurozael merged commit 999a37c into CloudSixteen:master Jul 11, 2013
@sanny-io
Copy link
Member Author

Will do.

@kurozael
Copy link
Member

Don't worry Duck I've already changed it for you and merged 👍

@sanny-io
Copy link
Member Author

Thanks ;D

sanny-io pushed a commit that referenced this pull request Jan 13, 2015
kurozael pushed a commit that referenced this pull request Dec 4, 2017
Merge from cloudsixteen/Clockwork
kurozael pushed a commit that referenced this pull request Aug 6, 2019
kurozael added a commit that referenced this pull request Aug 9, 2019
* Fixed `cw plysetgroup` not working from the server console
* Fixed setting a player's group causing a language error where #3 would be displayed
* Hints from the base gamemode will no longer be displayed at all
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants