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

2024/03/05 merge master #937

Merged
merged 337 commits into from
Mar 7, 2024
Merged

Conversation

DebugOk
Copy link
Contributor

@DebugOk DebugOk commented Mar 7, 2024

Untested, unfinished. Theres other commits still up for grabs and the trading post still needs to be partially copied due to missing out on a lot of content because of it.

Most notably, the shuttle ui refactor is missing, due to trading post FTL changes. I may still take care of that, but ehh.

I also dont believe tests will pass, and thats fine. If they do, great, otherwise I'll fix them whenever I feel like it.

Erisfiregamer1 and others added 30 commits March 5, 2024 09:49
Update food-component.ftl

why was I the only one that noticed it

(cherry picked from commit 39a2ff5)
(cherry picked from commit 786ecd9)
(cherry picked from commit 51caf3b)
…ial infected in zombie mode (#25529)

Remove the ability for command to be initial infected in zombie mode

(cherry picked from commit 5806401)
(cherry picked from commit c52820a)
* add

* ya

* add

(cherry picked from commit 9a4c10c)
(cherry picked from commit e52ebe0)
The collision layer check doesn't work and I don't have time to fix it.

(cherry picked from commit ec5edc3)
(cherry picked from commit 51acebd)
* Before SharedItemSystem

* Fixes mispredict, removed unused

* Fix physics

(cherry picked from commit 902198d)
(cherry picked from commit 6f3893a)
(cherry picked from commit a440ccf)
* Toggleable items now can change their shape depends on toggle state

* Update Content.Shared/Item/SharedItemSystem.cs

---------

Co-authored-by: metalgearsloth <[email protected]>
(cherry picked from commit 1a5f7c3)
* Juice that makes you Weh!

* Remove dupe

* updated the localization files

* try to fix error

(cherry picked from commit 1de102d)
* Added logic for ContainerSpawnPoint checks

* Improved with template function

* fixed nullable

* hehe

* hehe T?

* added type check before cast

* another nullable fix

* and another one

* return to old code (found typo)

* Code cleanup

---------

Co-authored-by: metalgearsloth <[email protected]>
(cherry picked from commit 87def40)
(cherry picked from commit 4613c9d)
IDK what this arg is for but seems okay with guns. Without it it always thinks the insertion can work.

(cherry picked from commit ad29dbe)
fix

(cherry picked from commit 7ba6765)
(cherry picked from commit 4e16034)
Turns out this is not what serverid was for... i guess you can find server familys with this so im not gonna remove it. Best we get for server name is the admin logs server name.

(cherry picked from commit 3ef4e83)
* Panic bunker fix

I admin

* tired

(cherry picked from commit b26acc9)
* Implemented contents display for dispenser UI

* Update Content.Client/Chemistry/UI/ReagentDispenserWindow.xaml.cs

Co-authored-by: Nemanja <[email protected]>

* Resolve the netent into a euid first

---------

Co-authored-by: Nemanja <[email protected]>
(cherry picked from commit 100ece2)
(cherry picked from commit 02be78b)
OnPiecePressed

(cherry picked from commit 7d6e4b1)
PJBot and others added 12 commits March 7, 2024 01:52
(cherry picked from commit 4228305)
Removed all obsolete, non-ECS method calls to BatteryComponent

(cherry picked from commit 2935e5b)
Added FitsInDispenser to FlaskBase

(cherry picked from commit 56454b6)
(cherry picked from commit 6da4009)
* fix(emag): Emagging a lock properly unlocks the component

Fixes #24860

* fix(emag): Emagging a lock no longer duplicates the unlock sound

(cherry picked from commit 370a79f)
* clown mask real

* Update masks.yml

(cherry picked from commit 2c2be09)
@github-actions github-actions bot added S: Needs Review Changes: C# Changes any cs files Changes: Documentation Changes any xml or md files Changes: Localization Changes any ftl files Changes: Map Changes: Sprite Changes any png or json in an rsi Changes: UI labels Mar 7, 2024
@DebugOk
Copy link
Contributor Author

DebugOk commented Mar 7, 2024

Yaml linter passed locally, thats the only test I ran

@deltanedas
Copy link
Member

i thought deltav still uses brig access for lawyers

@IamVelcroboy
Copy link
Member

i thought deltav still uses brig access for lawyers

No. we have a Lawyer access for that.

@DebugOk
Copy link
Contributor Author

DebugOk commented Mar 7, 2024

All tests are now green locally
image

@DebugOk DebugOk enabled auto-merge March 7, 2024 17:12
@DebugOk DebugOk merged commit 9b2d304 into DeltaV-Station:master Mar 7, 2024
10 checks passed
LaryNevesPR pushed a commit to LaryNevesPR/Estacao-Andromeda that referenced this pull request Jan 3, 2025
# Description
This significantly improves the quality of the language system by fixing
the mistakes I've made almost a year ago while developing it.

Mainly, this throws away the old half-broken way of networking in favor
of the component state system provided by RT. Language speaker comp is
now shared with SendOnlyToOwner = true, and its state is handled
manually.

In addition to that, this brings the following changes:
- UniversalLanguageSpeaker and LanguageKnowledge are now server-side
- DetermineLanguagesEvent and LanguagesUpdateEvent are now shared (so
that future systems can be built in shared, if needed)
- Everything now uses the ProtoId<LanguagePrototype> type instead of raw
strings (god, I hated those so much)
- The server-side language system now accepts Entity<T?> arguments
instead of EntityUid + T
- UniversalLanguageSpeaker is now based on DetermineEntityLanguagesEvent
and gets an Enabled field, which allows to turn it off. This may have
some use in the future.
- Some minor cleanup

<!--
TODO MEDIA

<details><summary><h1>Media</h1></summary>
<p>

![Example Media Embed](https://example.com/thisimageisntreal.png)

</p>
</details>

-->

# Changelog
No cl

---------

Co-authored-by: VMSolidus <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Documentation Changes any xml or md files Changes: Localization Changes any ftl files Changes: Map Changes: Sprite Changes any png or json in an rsi Changes: UI S: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.