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

Микроволокна #2679

Open
wants to merge 61 commits into
base: master
Choose a base branch
from

Conversation

SkaldetSkaeg
Copy link

@SkaldetSkaeg SkaldetSkaeg commented Mar 9, 2025

Описание PR

Адаптация ПР-а оффов, по добавлению волокон оставляемых хранилищами, так как они медлят

Медиа

Проверки

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

Изменения

🆑

  • add: Добавлены микроволокна, оставляемые инвентарями рюкзаков, курток и т д
  • add: Образцы и волокна пластинкой криминалистического сканера можно теперь собирать и через ПКМ

SkaldetSkaeg and others added 30 commits October 25, 2024 05:05
…izards#31625 (space-wizards#33447)

* Fix sprite layering upon strapping/unstrapping

* switch from strap to buckle events

* Use screen-local North for the checks instead

Build on top of PR space-wizards#31625, but I'm not wanting to just push to someone's older PR without a warning.

---------

Co-authored-by: themias <[email protected]>
* Slime plushie now squishes

* Remove useSound
* Grilled Cheese Prototype

* Typos

* Removed butter reagent to test flavour changes.

* Flavour corrections.

* CC-BY-SA-3.0 attribution

* Fixing meta.json

* I lost an hour of my life to a missing comma in meta.json

* Merge Conflict Rebase and Test Fixing

* Cook time to 10 seconds.

* Sandwich nutritional balance

* Grilled cheese sprite fixed, found the actual original source for copyright.
* Fix thrusters not dealing damage

* Fixes

* Update

* Update

* Update
* Made paramedic void suit into proper hardsuit (was softsuit before)
Added light to the paramedic void suit helmet
Added siren with light to the paramedic void suit

* Changed paramed helmet flashlight icon and textures

* fixed siren action icon

* BigAction

* updated icon-siren.png

* fixed not being able to add it to containers (like when it parented off eva suit)

* changed light to cyan, made it less brighter and reduced siren volume

* changed ambulancesiren.ogg to a different one (hopefully less hard on the ears)

* created ClothingHeadEVAHelmetWithLightBase in base_clothinghead.yml
replaced ambulancesiren.ogg (again, this time with better fading on the audio loop)
credited myself for the siren icon art
reverted parents to original (but kept it in hardsuits yml, since void suits are light hardsuits) - [ the adoption was revoked 😢 ]
fixed indentations
removed un-used sprite and sprite state

* final final siren sound change (for real this time?)

* fixed license type

* final FINAL final (for real this time, please?) ambulancesiren.ogg change

* ContainerContainerContainerContainerContainerContainerContainerContainerContainerContainerContainerContainerContainerContainer

* attributions updated (turns out, i have to give myself credit when i make stuff);
fixed identation;
moved action to Action/types.yml.

* hopefully fixed everything this time

* sync shenanigans

* final final final final final for real final ambulancesiren.ogg change pinky promise this time and if you don't like it, make a new PR yourself and offer a new sound

* typo

* done the suggested changes

* cleaning up some forgotten stuff :p

* oops, cleaned up a bit too much

* aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa

* Update Resources/Prototypes/Entities/Clothing/Head/eva-helmets.yml

* Update Resources/Prototypes/Entities/Clothing/Head/eva-helmets.yml

* Update attributions.yml

* evil web edit never again

---------

Co-authored-by: slarticodefast <[email protected]>
Co-authored-by: beck-thompson <[email protected]>
… spawning (space-wizards#35136)

* Filter NoStaticPriceAndStackPrice ents for StaticPriceComponent

* Remove redundant TryComp

* Use dependency shortcuts

* hol' up

* This filter is redundant

* Don't need EntityManager now
* Fix fax spamming

* Also prevent copy spam
…spesos, making burgers (space-wizards#33431)

* combine TrySpike into OnInteractUsing

* mark spike drink event as handled

* mark speso insertion event as handled

* mark food sequence event as handled

---------

Co-authored-by: slarticodefast <[email protected]>
* Make vote kicks ban the target's ip address

* Make it stop crashing my game
* Make vote kicks ban the target's ip address

* Make it stop crashing my game
* click clack

* fix quote marks

Co-authored-by: Hannah Giovanna Dawson <[email protected]>

---------

Co-authored-by: Errant <[email protected]>
Co-authored-by: Hannah Giovanna Dawson <[email protected]>
Simyon264 and others added 16 commits February 15, 2025 05:32
)

* add powered variable to vent components

* add checks for powered to vent systems

also corrected onpowerchanged methods to update powered arg.

* removed powered from components

* Use ApcPowerReceieverComponent for power state.

* removed unneeded code from OnPowerChanged

* document what enabled is used for in components

* only you can prevent oopsie daisies.

* add check for powered in OnGasVentPumpUpdated

* apcPowerReceiverComponent BEGONE

* CODE RED EVERYTHINGS ON FIRE wait we're fine now.
@github-actions github-actions bot added Changes: UI Изменение затронуло файлы ".xaml" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: C# Изменение затронуло файлы ".cs" labels Mar 9, 2025
@SkaldetSkaeg SkaldetSkaeg requested a review from Kirus59 March 9, 2025 10:54
@SkaldetSkaeg
Copy link
Author

@stalengd @Kirus59 Мне как быть с оригинальными файлами, которые у оффов в рассмотрении?
Нужно ли переносить компоненты и локали в отдельные файлы, в ожиданнии того, что могу вобще никогда не принять?

@Ady4ik
Copy link

Ady4ik commented Mar 9, 2025

Я прочитал микроволновка..

@stalengd
Copy link
Collaborator

stalengd commented Mar 9, 2025

@SkaldetSkaeg плохо что ты потянул за собой мастер оффов

@SkaldetSkaeg
Copy link
Author

@SkaldetSkaeg плохо что ты потянул за собой мастер оффов

честно сказать, пока хз, что с этим делать и насколько это критично. Ждал что будет апстрим, но не помогло особо

Copy link
Collaborator

@UrPrice UrPrice left a comment

Choose a reason for hiding this comment

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

Только отгрузи лишнее в виде мастера офов и оставь чисто свою фичу.
Нам апстрим преждевременный выйдет боком из-за изменений, что там вышли

@SkaldetSkaeg
Copy link
Author

Только отгрузи лишнее в виде мастера офов и оставь чисто свою фичу. Нам апстрим преждевременный выйдет боком из-за изменений, что там вышли

окей, переделать, после такого будет не сложно, после фикса всех проблем

@SkaldetSkaeg
Copy link
Author

Только отгрузи лишнее в виде мастера офов и оставь чисто свою фичу. Нам апстрим преждевременный выйдет боком из-за изменений, что там вышли

главый вопрос -- сколько ждать оффов чтобы сказать, что они PR примут точно не скоро/вобще не примут? нужно ли переносить всё в папки SS220?

@UrPrice
Copy link
Collaborator

UrPrice commented Mar 10, 2025

главый вопрос -- сколько ждать оффов чтобы сказать, что они PR примут точно не скоро/вобще не примут? нужно ли переносить всё в папки SS220?

Ну такую тенденцию мы уже делали с касетным диктофоном, надо её просто продолжить. @stalengd тут лучше помнит/знает

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: UI Изменение затронуло файлы ".xaml"
Projects
None yet
Development

Successfully merging this pull request may close these issues.