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

Modular arrows #906

Merged
merged 10 commits into from
Feb 19, 2025
Merged

Modular arrows #906

merged 10 commits into from
Feb 19, 2025

Conversation

Nimfar11
Copy link
Contributor

@Nimfar11 Nimfar11 commented Feb 16, 2025

About the PR

Arrows are now made with a modular system too.
Теперь стрелы тоже делаются с помощью модульной системы.

Media

изображение

Copy link
Contributor

github-actions bot commented Feb 16, 2025

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

Resources/Textures/_CP14/Objects/ModularTools/Rod/arrow.rsi

State Old New Status
icon Added
inhand-left Added
inhand-right Added
tail Added

Resources/Textures/_CP14/Objects/ModularTools/Tip/arrow.rsi

State Old New Status
icon Added
inhand-left Added
inhand-right Added
solution1 Added

Edit: diff updated after 66a2e4d

@Nimfar11
Copy link
Contributor Author

Странная ошибка, это из-за того что нет sourcePart ?

@Deserty0
Copy link
Contributor

Странная ошибка, это из-за того что нет sourcePart ?

Field "targetSlot" not found
Буквально самая понятная ошибка которую линтер только способен выдать

@Deserty0
Copy link
Contributor

Я так посмотрел, я вообще не нашёл ни одного targetSlot ни в существующих прототипах, ни в системе

@TheShuEd
Copy link
Contributor

Буквально вчера ночью обновилось просто

@TheShuEd
Copy link
Contributor

теперь это slots: и список

@Nimfar11
Copy link
Contributor Author

Я так посмотрел, я вообще не нашёл ни одного targetSlot ни в существующих прототипах, ни в системе

а у меня они были, вот и в непонятках был

@Nimfar11
Copy link
Contributor Author

Сами стрелы не удалял, потому что они могут быть где-то замаплены, чтобы ничего не поломалось там.

@TheShuEd
Copy link
Contributor

Сделай как с модульным готовым оружим - сделай пресет стандартной стрелы, которая автоматически собирается из модулей (посмотри по примерам), и в миграции просто свапни старые стрелы на новые.

@Nimfar11
Copy link
Contributor Author

Сделай как с модульным готовым оружим - сделай пресет стандартной стрелы, которая автоматически собирается из модулей (посмотри по примерам), и в миграции просто свапни старые стрелы на новые.

Новые стрелы через пресет есть, я их закинул в колчаны уже. А вот миграцию да, можно

@@ -82,6 +82,54 @@
types:
Blunt: 0

- type: entity
Copy link
Contributor

Choose a reason for hiding this comment

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

я бы вынес все что касается стрел в отдельный файл. Tip как наконечник звучит как что-то, что может быть наконечником не только стрелы. Я бы назвал ArrowTip?
Grip = рукоять, у стрел рукояти нет, там древко

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Вот теперь точно поправил.

Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@TheShuEd TheShuEd merged commit 88462e5 into crystallpunk-14:master Feb 19, 2025
13 checks passed
@Nimfar11 Nimfar11 deleted the modular-arrow branch February 19, 2025 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants