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

FixNuke #2585

Closed
Closed

Conversation

Helm4142
Copy link

Описание PR

Задача --> https://github.com/orgs/SerbiaStrong-220/projects/26?pane=issue&itemId=98377136
Взорвал аванпост НТ, саботажа больше не будет.
Медиа

Проверки

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

Изменения

🆑 Helm

  • fix: Исправлена выдача вещей Оперативникам!

@github-actions github-actions bot added the Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" label Feb 19, 2025
@Helm4142 Helm4142 changed the title Rukojopiki FixNuke Feb 19, 2025
@UrPrice
Copy link
Collaborator

UrPrice commented Feb 19, 2025

А кома?

@Helm4142
Copy link
Author

А кома?

Я всё протестил, не знаю как и почему но кома подсосалась сама, там чёт через медика идёт, странная хрень в общем

Copy link

@Kirus59 Kirus59 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Пробелы в разметке массивов никак не влияют на извлечение их значений (я про то, что ты зачем то добавил пробелы в полях prototypes)
  2. Ошибку вероятно писали о ролях выдаваемых геймрулом ЯО, там лодаут прописывается отдельно, так что твои текущие изменения на это никак не влияют

@@ -39,7 +39,7 @@
name: syndicate medic
components:
- type: Loadout
prototypes: [SyndicateReinforcementMedic]
prototypes: [ SyndicateOperativeMedicFull ] #SS220-FixNuke
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это медик с подкрепы, у него и должно быть слабое снаряжение

@Kirus59
Copy link

Kirus59 commented Feb 19, 2025

В общем, это сломалось наследование у StartingGearPrototype.
Через прототипы это никак не починить (можно конечно просто продублировать в каждый прототип нужные позиции, но лучше починить наследование, чем делать такие костыли).

@Kirus59
Copy link

Kirus59 commented Feb 19, 2025

#2593

@Kirus59 Kirus59 closed this Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants