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

Fix starting gear inharitance #2593

Merged
merged 2 commits into from
Feb 19, 2025

Conversation

Kirus59
Copy link

@Kirus59 Kirus59 commented Feb 19, 2025

Описание PR

Исправлено наследование поля Equipment в прототипе StartingGearPrototype

Проверки

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

Изменения

🆑

  • fix: Исправлено отсутствие некоторой одежды у ролей ЯО при спавне

@Kirus59 Kirus59 requested a review from stalengd February 19, 2025 19:20
@github-actions github-actions bot added Changes: C# Изменение затронуло файлы ".cs" Status: Needs Review labels Feb 19, 2025
@Kirus59 Kirus59 mentioned this pull request Feb 19, 2025
5 tasks
@stalengd
Copy link
Collaborator

@Kirus59

  1. Что оно исправляет?
  2. Запускал в Release конфигурации?

@Kirus59
Copy link
Author

Kirus59 commented Feb 19, 2025

  1. Что оно исправляет?
  2. Запускал в Release конфигурации?
  1. Теперь при наследовании если у текущего прототипа поле Equipment отличается от родительского - то оно не полностью заменяется, а объединяется по ключам (по сути происходит замена значений ключей, вместо всего поля)
    Раньше это так и работало, однако я не нашел где это было записано.
    Также не нашел как можно переписать этот прототип в PrototypeManager не трогая движок, поэтому сделал так.

  2. Да, запускал, работает как и задуманно

@stalengd stalengd merged commit 0be4897 into SerbiaStrong-220:master Feb 19, 2025
13 checks passed
@Kirus59 Kirus59 deleted the fix-gear-inheritance branch February 19, 2025 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants