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

NewBarricade #696

Merged
merged 56 commits into from
Feb 2, 2024
Merged

NewBarricade #696

merged 56 commits into from
Feb 2, 2024

Conversation

EstKemran
Copy link

@EstKemran EstKemran commented Jan 31, 2024

Описание PR

https://discord.com/channels/1097181193939730453/1200711354013716581
Баррикады из трех материалов. Баррикады обычные, направленные и напольные. Первые два вида имеют компонент Occluder, последний нет. Отличаются стоимостью материалов и прочностью. Эти баррикады можно строить на стене/окне, а так же разрушить с помощью ближнего боя (В отличии от оффских). Эти баррикады в первую очередь защищают то, к чему они прикреплены(В отличии от оффских).

Изменены границы направленной деревянной баррикады от оффов. Убран компонент, запрещающий строить баррикады на окнах/стенах.

Медиа

Content Client_AwPDklMAJN
Content Client_BmLyjlnzn7

Проверки

  • [+] PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • [+] Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • [+] Я запускал локальный сервер со своими изменениями и всё протестировал.
  • [+] Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

  • add: Добавлено 9 новых баррикад
  • tweak: Деревянные баррикады теперь можно строить на окнах/стенах
  • tweak: С направленной деревянной баррикадой теперь можно взаимодействовать

@github-actions github-actions bot added Changes: Sprites Изменение затронуло файлы ".rsu / .png" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" labels Jan 31, 2024
Copy link

github-actions bot commented Jan 31, 2024

RSI Diff Bot; head commit a9dba49 merging into 6caa778
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/SS220/Structures/barricades.rsi

State Old New Status
barricade_plasteel Added
barricade_plasteel_directional Added
barricade_plasteel_floor Added
barricade_steel Added
barricade_steel_directional Added
barricade_steel_floor Added
barricade_wooden Added
barricade_wooden_directional Added
barricade_wooden_floor Added

Edit: diff updated after a9dba49

@Kit0vras
Copy link

image
тут ты добавил 2 одинаковых файла, а самих рецептов нет

в отредактированные айди я подописывал 220, потому что не знаю, могут ли они совпасть с айди оффовскими баррикад, если нет - то 220 тоже можно убрать

@EstKemran
Copy link
Author

@Kit0vras Добавил нужный прототип. Проверь его пожалуйста. У меня с ним что-то не получилось.

EstKemran and others added 22 commits January 31, 2024 18:49
EstKemran and others added 16 commits February 2, 2024 13:54
@TheArturZh
Copy link
Member

image

@EstKemran
Copy link
Author

EstKemran commented Feb 2, 2024

@TheArturZh Посмотри коммит update barricadesteel.yml, и если все нормально, запусти пожалуйста тест снова

@EstKemran
Copy link
Author

АААААА! Наконец-то он заработал. Огромное вам спасибо, Kit0vras, TheArturZh.

@TheArturZh TheArturZh merged commit a015502 into SerbiaStrong-220:master Feb 2, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: Sprites Изменение затронуло файлы ".rsu / .png"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants