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

[Hotfix] Inventory slots dissapear at roundstart #93

Merged

Conversation

TokenStyle
Copy link

Basically reverts disabling feature of limbs cutting

Описание PR

Исправил откатив фичу с исчезанием слотов при отрубании конечностей. Эта фича багованная нужно переписать. Сегодня будет пофикшено, но чтобы игроки не страдали, откатил эту фичу со слотами. Слоты будут левитировать, но зато игроки не будут страдать.

Почему / Баланс

Ссылка на ветку

https://discord.com/channels/919301044784226385/1309856788346044416/1311175685599006730

Технические детали

Система слотов ломается из-за щиткода на стороне оффов. Я просто закомментировал фичу //slot.Disable = isDisabled; в InventorySystem.Slots.cs. Вот подробности про багу:

I think I moved everything related to slot disabling/enabling to a centralized function called ChangeSlotState or something?
its really messy since inventorycomponent and inventoryslotscomponent are not the same.
and inventory slots arent networked since it uses a compreg

Медиа

image

Критические изменения

Список изменений

🆑

  • fix: Теперь раундстартом доступны все слоты под перчатки, наушники, ботинки, головной убор, когда ранее они с шансом были недоступны.
  • fix: Теперь отрубание конечностей не убирает возможность взаимодействия с слотом под предмет, например отрубание ног позволяет надеть ботинки.

Basically reverts disabling feature of limbs cutting
@TokenStyle
Copy link
Author

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

@Tornado-Technology Tornado-Technology merged commit e2161be into space-syndicate:master Nov 27, 2024
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: Needs Review ПРу необходим ревью S: Untriaged size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants