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

'25 February upstream merge #2897

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

Conversation

whatston3
Copy link
Contributor

@whatston3 whatston3 commented Feb 10, 2025

About the PR

Merges vs upstream's 'stable' branch at 11e5d59171739c11c8b1fe2271b3aceca8ff03ed.

At the time of writing, I've only (poorly) resolved merge conflicts to get this to a draftable state. Will require work and testing.

Items of note:

  • emag change (what will the state of the demag be?)
  • weapon component changes?
  • latejoin/humanoid profile edits?
  • syndie radio implant
  • guidebook entries for engineering (station repair)

Why / Balance

It is time.

How to test

Very carefully. Points of interest:

  • Cargo bounty console has history - do we care about the pirate console? Not right now.
  • There’s a new build fork ID and version CVAR. Ideally we set these.
  • Is the emote menu fine? For harpies? Yup.
  • Are criminal records functional? Yes.
  • What is “GhostRoleRadioMessage”? Is it working? Think so.
  • Crew monitoring window should be in world coords. Test it works. Seems to.
  • EyeCursorOffset is new. Should we add it to the SVT/Mosin? Maybe later, fine for now.
  • ShipyardRCD: menu works? yes
  • Contraband test: should we extend this? No need.
  • GuidebookDataAttribute: does this need to be extended to our gas pipe changes? Eventually.
  • Silicon laws - emag disabled for now.
  • Test the demag on everything. Most things that are emaggable are demaggable. Generally good
  • BuckleSystem changed, are vehicles/chairs/etc. busted? Not obviously. Did the toilet teleport bug get fixed? Yes.
  • LoadoutWindow: named loadouts are here - pirates have pseudonyms, borgs should show up and have a selectable name. Done.
  • The window with only the selectable name is a bit small. Fixed.
  • Multiple storage windows open: how do we enable this (if we can) Not currently. Soon™
  • Air alarm logging for the bidirectional gas pressure pump stuff? Not right now.
  • Virtual item changes: map onto the borg hand placeholder? Not currently needed, works fine.
  • Are research servers functioning as usual? Seems like it.
  • Unrevivable now has a reason. Should unclonable work the same way? Done.
  • The publish-testing workflow was disabled. Do we need it? Not currently.
  • Check seed labels for Frontier sprites (bee jar, coffee, spesos, pear) Paper seems alright, jar's a little weird.
  • Tactical maid light is functional? Does the action sprite look alright? They're fine.
  • Do all of the ammo boxes work? Any missing sprites? Think we're good.

Media

Requirements

Breaking changes

Changelog

🆑

  • add: Pirates can now set pseudonyms in character config, cyborgs can set their names.
  • tweak: The advanced captain's laser has been moved from a printable item to expedition loot.
  • fix: Goblins should no longer teleport back into toilets.
  • tweak: Random clothes with pants and skirts are available in loadouts and the Clothesmate.
  • fix: Bluespace crystal stacks should all be tiny.
  • tweak: The tactical maid hardsuit now reduces movement speed by 25% instead of 55%.
  • remove: Silicons can no longer be emagged.
  • tweak: All contraband items with visible status now have their class stated.
  • fix: Wooden barrels no longer show an error with a paper label placed on them.

ShadowCommander and others added 30 commits February 6, 2025 16:14
* Test entered tree fix

* Use Opened override

* Fix keyboard focus on window open for DialogWindow

This affects Phone, Pray, Rename, and multiple other admin verbs.

* Clean up
* cleanup not needed locale string

* Update Resources/Prototypes/Actions/types.yml

---------

Co-authored-by: slarticodefast <[email protected]>
* Remove custom character description after using DNA scrambler

* Mark grammar as dirty when updating identity

* Update Content.Server/Implants/SubdermalImplantSystem.cs

---------

Co-authored-by: slarticodefast <[email protected]>
fixed control button - a single line of code
* Buff the Combat Bakery Kit uplink item

* whitespace
Switches over all extant uses of TransformComponent.AttachToGridOrMap to use SharedTransformSystem.AttachToGridOrMap
…095)

* add LatheRecipePackPrototype

* change Lathe and EmagLathe to use packs

* add AddRecipesFromPacks helper to SharedLatheSystem

* update lathe logic to work with packs and clean up some stuff

* migrate individual recipes to recipe packs

* update client

* remove node/artifact scanner from techs

* :trollface:

* fix test and make it include emag recipes

* add test that every dynamic recipe must be researched

* pro

* fix

* fix

* fix all tests, genuinely good test i wonder who made it

* add unused uranium and incendiary drozd mags to tech and lathe

* add recipes

* add incendiary prototype

* undo some changes

* troll

* :trollface:

* true

Co-authored-by: pathetic meowmeow <[email protected]>

* shitmed real

Co-authored-by: pathetic meowmeow <[email protected]>

* update funny test

* :trollface:

* :trollface:

---------

Co-authored-by: deltanedas <@deltanedas:kde.org>
Co-authored-by: pathetic meowmeow <[email protected]>
Update plasma.yml

Co-authored-by: jbox1 <[email protected]>
* Add paper door sound

* Update attributions yml

* Update material_doors.yml

* Update attributions.yml
* wooden grip on antique laser, changes description to match.

* darker grip

* update meta.json

* redesigned shape similar to ps3moira, edited mag to match, changed lights colors to match each other

* fixing an only SLIGHTLY catastrophic fuckup with an experimental inhand sprite that I didn't mean to save and sleepily added to the previous commit

* I DID IT AGAIN

* red kammerer bottom text

* killed the cap laser things (hopefully)

---------

Co-authored-by: gaussiarson-admin <[email protected]>
The flammable effect does it but we don't do it here for reasons. Sometimes this heisentests.
* Added back warden's enforcer, added spare mk58 magazines to blue armory.

* Various QOL.
* Updated armory fills, fixed a few connection errors, and removed the original sin

* Added a box of .35 magazines, corrected the starting state of the switches
* allow paper labels on seeds

* added the components required by PaperLabel to the SeedBase

* localize insert/eject for PaperLabel

* visualize the paper label on seeds

* new paper layer for seeds

* eject the seed label on use

* different looking paper label

* forgot to turn on a layer
@whatston3 whatston3 added S: Needs Review This PR is awaiting reviews and removed S: Awaiting Changes This PR has changes that need to be made before merging S: DO NOT MERGE labels Feb 25, 2025
@whatston3
Copy link
Contributor Author

DB migration tested and working well with Postgres 17 and Sqlite.

Lathe packs should probably get a review, but for now, the set of recipes included everything that was previously printable, though Frontier-specific lathes generally have one pack with all the recipes rather than sharing packs where appropriate.

Changelog should be appropriate, and the demag should be more useful than ever. Think this is alright to proceed.

@github-actions github-actions bot added the S: Merge Conflict This PR has conflicts that prevent merging label Feb 26, 2025
Copy link
Contributor

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

@github-actions github-actions bot removed the S: Merge Conflict This PR has conflicts that prevent merging label Feb 26, 2025
@github-actions github-actions bot added the S: Merge Conflict This PR has conflicts that prevent merging label Feb 26, 2025
Copy link
Contributor

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

@github-actions github-actions bot removed the S: Merge Conflict This PR has conflicts that prevent merging label Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Audio C# DB Migration The PR touches database migrations - merge with caution and test thoroughly Docs Improvements or additions to documentation FTL Map-Shuttle Map - Shuttle S: Needs Review This PR is awaiting reviews size/M Sprites UI YML
Projects
None yet
Development

Successfully merging this pull request may close these issues.