update {get} UnturnedPlayer.Color #82
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.
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мя циклами или с сохранением значения