-
Notifications
You must be signed in to change notification settings - Fork 9
update {get} UnturnedPlayer.Color #82
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
Conversation
Hello, I looked at the code, everything seems fine, I don’t quite understand what it does)
|
That's cool, but maybe such a change could break backward compatibility.Это круто, но возможно такое изменение может сломать обратную совместимость |
And wait, it's off by defaultА стоп, по умолчанию это выключено |
|
там же условие через if, я думаю ничего не ломается вопрос только в том что возможно придется конфиг удалить чтобы добавился этот bool |
|
Да это основная проблема) Но думаю можно пережить это |
I would accept this, but perhaps we will wait for the main developer)Я бы принял это, но пожалуй дождёмся главного разработчика) |
да чето напутал я
Config is fine, it should auto add the Property, with the default value (i.e. as false) |
sunnamed434
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made well, looks good to me, its also nice that you made an option EnableUnturnedPlayerColorFromPriorityGroup and the feature can be disabled/enabled.
I'm sure many folks will find this feature useful

I noticed that the priority is not used at all
add EnableUnturnedPlayerColorFromPriorityGroup (def is false) in config
select the desired option by the community - 2 cycles or with the priority value saved
//
я заметил, что приоритет вообще нигде не используется
добавить EnableUnturnedPlayerColorFromPriorityGroup
выбрать из двух вариантов - проход 2мя циклами или с сохранением значения