OnyxBay
OnyxBay copied to clipboard
feat(loadout): adding jobs winter coats
Добавили уже существующие шубы отделов из билда в лодаут, чтобы игроки могли пользоваться ими на здоровье, и они не валялись как мёртвый груз. Также добавили новое значение для gear_tweak'а, позволяющее создавать списки предметов, ориентирующихся на конкретную профессию.
Картинки
Чейнджлог
🆑 ilyadobr & rufuszero
rscadd: Шубы отделов доступны в лодауте
/🆑
- [x] Pull Request полностью завершен, мне не нужна помощь чтобы его закончить.
- [x] Я внимательно прочитал все свои изменения и багов в них не нашел.
- [x] Я запускал сервер со своими изменениями локально и все протестировал.
- [x] Я ознакомился c Guide to Contribute.
Картинка действительно страшная. Есть мысль следующая.
Добавить опцию Departmental winter coat которая будет автоматически назначать нужную шмотку.
Попробую потыкаться на днях с этим, если ничего не выйдет, буду просить подкрепление и помощи 😞
@BaraBarax без проблем, поправим, только можно уточнить, зачем эти ньюлайны нужны? Есть какой-то док по кодстайлу где с подобными предпочтениями мейнтейнеров и их причинами можно ознакомиться? Или это просто "так заведено"?
@rufuszero Гипотетически всё должно быть в Guide to Contribute, но написано ли там про ньюлайны - хз
@BaraBarax
Так я и смотрю в Guide to Contributing, я с ним полностью ознакомился перед заливкой своего кода.
Просто не совсем понятно откуда такие моменты узнавать заранее чтоб не тратить время мейнтейнеров и контрибьюторов на ревьюшки буквально с ньюлайнами.
Спасибо что указал на это. Добавлю, недоразумение вышло.
В связи с тем, что ПР не может пройти в билд, поскольку спрайты не соответствуют актуальному состоянию спрайтов билдов, ПР снова уходит в драфт, пока я не найду спрайтеров, которые допилят спрайты до актуального состояния, или я самолично этим не займусь.
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Полярку вернули, шубы больше не мёртвый груз. Редунтант ПР.