Skip to content

Granate#203

Open
AustraloPiteck wants to merge 6 commits into
masterfrom
Granate
Open

Granate#203
AustraloPiteck wants to merge 6 commits into
masterfrom
Granate

Conversation

@AustraloPiteck

Copy link
Copy Markdown
Collaborator

Add Granate Mechanic

Comment thread Assets/Failsafe/Scripts/Granates/Grenade.cs Outdated
Comment thread Assets/Failsafe/Scripts/Granates/Grenade.cs Outdated
Comment thread Assets/Failsafe/Scripts/Granates/Grenade.cs Outdated

@captaincolonelfox captaincolonelfox left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Выглядит неплохо

Нужно ребейзнуть на мастер, реализовать новый интерфейс Item в Grenade (добавить ItemUseResult), создать префаб (пока что просто с цилиндром), закинуть его на сцену DataCenter для тестов

Скорее всего, чтобы это заработало, нужно будет ещё повозиться с LifetimeScope, потому что через него добавляются итемы и их скрипты в Player (если с этим будут проблемы, то можно будет смержить и без префаба, но нужна будет помощь Эдуарда, чтобы подключить это к системе PlayerHandsSystem)

@trofimoved

Copy link
Copy Markdown
Collaborator

Скорее всего, чтобы это заработало, нужно будет ещё повозиться с LifetimeScope, потому что через него добавляются итемы и их скрипты в Player

Это нужно делать только для предметов, который инжектят что то из PlayerLifetimeScope. В данном случае добавлять гранату в скоуп не нужно.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants