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

Upstream2107 (Требуется тестирование) #133

Merged
merged 102 commits into from
Jul 26, 2024

Conversation

Schrodinger71
Copy link
Collaborator

@Schrodinger71 Schrodinger71 commented Jul 21, 2024

Описание PR

Changelog.yml

- author: ShadowCommander
  changes:
  - message: Players can now use melee attacks and shoves while dragging an entity
      in their active hand.
    type: Tweak
  id: 6900
  time: '2024-07-11T04:48:00.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29703
- author: Cojoke-dot
  changes:
  - message: You can no longer shoot out of crates with guns
    type: Fix
  id: 6901
  time: '2024-07-11T05:14:49.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/28961
- author: Cojoke-dot
  changes:
  - message: The Spray Painter can now be used to paint glass airlocks to look like
      regular glass airlocks.
    type: Tweak
  id: 6902
  time: '2024-07-11T05:33:20.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29869
- author: MFMessage
  changes:
  - message: Picking a ghost role as an admin will now deadmin.
    type: Fix
  id: 6903
  time: '2024-07-11T05:53:15.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29790
- author: Winkarst-cpu
  changes:
  - message: Admin notes popups are now more readable.
    type: Tweak
  id: 6904
  time: '2024-07-11T14:03:22.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29909
- author: jonathanargo
  changes:
  - message: Muskets are now wieldable.
    type: Tweak
  id: 6905
  time: '2024-07-12T09:16:21.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29910
- author: themias
  changes:
  - message: Notice boards can now be built on walls
    type: Fix
  id: 6906
  time: '2024-07-12T09:18:32.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29851
- author: slarticodefast
  changes:
  - message: Stun batons, stun prods and banana cream pies now fly like other throwing
      weapons when thrown.
    type: Fix
  id: 6907
  time: '2024-07-12T09:19:24.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29883
- author: Plykiya
  changes:
  - message: Dropping an item while in a container now places the item in the container.
    type: Fix
  id: 6908
  time: '2024-07-12T09:24:08.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29900
- author: coffeeware, slarticodefast
  changes:
  - message: Fixed items thrown very fast not being in the air. In particular this
      fixes the pneumatic cannon.
    type: Fix
  - message: Buffed base hand throwing speed by 10% to be more similar to before the
      recent throwing changes.
    type: Tweak
  id: 6909
  time: '2024-07-12T10:32:47.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29935
- author: themias
  changes:
  - message: Timers can now be deconstructed
    type: Fix
  id: 6910
  time: '2024-07-12T11:38:59.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29917
- author: jonathanargo
  changes:
  - message: Hard hat icon sprites are now centered correctly.
    type: Fix
  id: 6911
  time: '2024-07-12T15:00:40.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29953
- author: lzk228
  changes:
  - message: Secure windoors now use reinforced glass damage modifier.
    type: Fix
  id: 6912
  time: '2024-07-13T04:03:16.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29941
- author: Tayrtahn
  changes:
  - message: Antag objective total difficulty is now properly capped.
    type: Fix
  id: 6913
  time: '2024-07-13T04:14:30.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29830
- author: EmoGarbage404
  changes:
  - message: Added wristwatches for telling time.
    type: Add
  id: 6914
  time: '2024-07-13T06:09:19.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29550
- author: TheShuEd
  changes:
  - message: Added diamonds ore!
    type: Add
  - message: Diamonds can now be sold at a bargain price.
    type: Add
  id: 6915
  time: '2024-07-13T12:15:57.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/25750
- author: coffeeware
  changes:
  - message: Lizards will no longer lose their snouts when equipping head bandanas
    type: Fix
  id: 6916
  time: '2024-07-14T02:59:45.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29979

Admin.yml

- author: Chief-Engineer
  changes:
  - message: Baby jail no longer prevents accounts with no prior connections from
      connecting
    type: Fix
  id: 45
  time: '2024-07-11T05:14:01.0000000+00:00'
  url: https://github.com/space-wizards/space-station-14/pull/29896

Проверки

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

Изменения

🆑 Шрёдька

  • tweak: Подтянуты изменения с корвакса, 21.07.24.

ShadowCommander and others added 30 commits July 11, 2024 14:48
* Make VirtualItem not block attacking

* Remove unneeded usings
* Revert "Revert Baby Jail (#29891)"

This reverts commit 24a2866.

* the fix
* Fix the ability to shoot out of crates

* Makes it check what inventory the player is in

* use IsEntityOrParentInContainer

* Fix Issues Github had

* gaahhh... Prevents lasers from being shot out of crates

* gaahhh... Prevents lasers from being shot out of crates

* Fix laser?

* hmmm... this is better looking I think?

* Uncook indentation

* Rerun tests?
* @Forcibly deadmins you

* Added checks for AdminDeadminOnJoin
* item toggle refactoring and some new systems

* add ToggleClothing component/system

* unhardcode magboots gravity logic

* make magboots and speedboots use ItemToggle and stuff

* remove now useless clothing components

* update client/server magboots systems

* add note to use ItemToggledEvent in ToggleActionEvent doc

* refactor PowerCellDraw to use ItemToggle for ui open/close control

* add TryUseCharges, refactor charges system

* update magboot trigger code

* make borg use ItemToggle, network SelectedModule instead of now removed Activated

* add AccessToggle for borg

* the giga ninja refactor

* update ninja yml

* update ItemToggle usage for some stuff

* fix activatableui requires power

* random fixing

* yaml fixing

* nuke ItemToggleDisarmMalus

* make defib use ItemToggle

* make things that use power not turn on if missing use charge

* pro

* fix sound prediction

* bruh

* proximity detector use ItemToggle

* oop

* big idiot syndrome

* fix ninja spawn rule and make it generic

* fix ninja spawn rule yml

* move loading profiles into AntagLoadProfileRule

* more ninja refactor

* ninja yml fixes

* the dreaded copy paste ops

* remove useless NinjaRuleComponent and ue AntagSelection for greeting

* fix invisibility

* move IsCompleted to SharedObjectivesSystem

* ability fixes

* oop fix powercell instantly draining itself

* sentient speedboots gaming

* make reflect use ItemToggle

* fix other test

* loadprofilerule moved into its own pr

* remove conflict with dragon refactor

* remove all GenericAntag code from ninja

* )

* probably

* remove old enabled

* great language bravo vince

* GREAT LANGUAGE

* who made this language

* because it stinks

* reparent blood-red magboots to magboots probbbly works

* most of the review stuff

* hasGrav doesnt mean what i thought it did

* make health analyzer use itemtoggle, not fail test

* fix mag/speed boots being wacky

* UNTROLL

* add ItemToggle to the random health analyzers

* a

* remove unused obsolete borg func

* untrolling

* :trollface:

* fix test

* fix

* g

* untroll

---------

Co-authored-by: deltanedas <@deltanedas:kde.org>
Fix AtmosDeviceSystem debug assertion Heisenbug
* Better admin note popup text visibility

* Bring buttons closer to the border
* Update cluster evac

* forgot tiny fans

* the change
* Update FloorTileSystem to use EntityUid

* missed a variable

---------

Co-authored-by: plykiya <[email protected]>
* Made musket wieldable.

* Updated musket rsi meta.json copyright.

* Copyright adjustment again.
* make stun batons be thrown like a throwing weapon

* stun prods too

* pies too
Update MapScreen.xaml.cs

Co-authored-by: plykiya <[email protected]>
* Items droped in containers will end up in containers

* Adds integration test for dropping entity while inside container

* comment

* comment

* trim the diff

---------

Co-authored-by: plykiya <[email protected]>
throwing system hotfix
@Schrodinger71 Schrodinger71 self-assigned this Jul 21, 2024
@github-actions github-actions bot added Changes: UI Изменение интерфейса Changes: Sprites Изменение спрайтов Changes: Map Изменение карт Changes: Localization Изменение локализации labels Jul 21, 2024
@Schrodinger71
Copy link
Collaborator Author

Schrodinger71 commented Jul 21, 2024

Кайф, выскочили ошибки в системе с языками
хммм PowerCell

Error: /home/runner/work/space_station_ADT/space_station_ADT/Content.Server/ADT/Language/TranslatorSystem.cs(165,24): error CS1061: 'PowerCellSystem' does not contain a definition for 'SetPowerCellDrawEnabled' and no accessible extension method 'SetPowerCellDrawEnabled' accepting a first argument of type 'PowerCellSystem' could be found (are you missing a using directive or an assembly reference?) [/home/runner/work/space_station_ADT/space_station_ADT/Content.Server/Content.Server.csproj]
Error: /home/runner/work/space_station_ADT/space_station_ADT/Content.Server/ADT/Language/TranslatorSystem.cs(174,24): error CS1061: 'PowerCellSystem' does not contain a definition for 'SetPowerCellDrawEnabled' and no accessible extension method 'SetPowerCellDrawEnabled' accepting a first argument of type 'PowerCellSystem' could be found (are you missing a using directive or an assembly reference?) [/home/runner/work/space_station_ADT/space_station_ADT/Content.Server/Content.Server.csproj]
Error: /home/runner/work/space_station_ADT/space_station_ADT/Content.Server/ADT/Language/TranslatorSystem.cs(191,20): error CS1061: 'PowerCellSystem' does not contain a definition for 'SetPowerCellDrawEnabled' and no accessible extension method 'SetPowerCellDrawEnabled' accepting a first argument of type 'PowerCellSystem' could be found (are you missing a using directive or an assembly reference?) [/home/runner/work/space_station_ADT/space_station_ADT/Content.Server/Content.Server.csproj]

Меняем SetPowerCellDrawEnabled на новый метод SetDrawEnabled
@Schrodinger71 Schrodinger71 added the Не Мерджить Не МЕРДЖИТЬ НИ В КОЕМ СЛУЧАЕ label Jul 21, 2024
@Schrodinger71
Copy link
Collaborator Author

Хм, теперь такая

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
  Failed NoStaticPriceAndStackPrice [1 s]
  Error Message:
   Multiple failures or warnings in test:
  1) SERVER: 1,481s [ERRO] system.item_toggle: Can't resolve "Content.Shared.Item.ItemToggle.Components.ItemToggleComponent" on entity переводчик вульпканинов (11799/n11799, VulpTranslator)!
   at System.Environment.get_StackTrace()
   at Content.Shared.Item.ItemToggle.ItemToggleSystem.TryDeactivate(Entity`1 ent, Nullable`1 user, Boolean predicted) in /home/runner/work/space_station_ADT/space_station_ADT/Content.Shared/Item/ItemToggle/ItemToggleSystem.cs:line [13](https://github.com/AdventureTimeSS14/space_station_ADT/actions/runs/10027377105/job/27712969172?pr=133#step:10:14)8
   at Content.Server.PowerCell.PowerCellSystem.OnDrawCellChanged(EntityUid uid, PowerCellDrawComponent component, PowerCellChangedEvent args) in /home/runner/work/space_station_ADT/space_station_ADT/Content.Server/PowerCell/PowerCellSystem.Draw.cs:line 63

@Schrodinger71 Schrodinger71 changed the title Upstream2107 Upstream2107 (Требуется тестирование) Jul 21, 2024
@Schrodinger71
Copy link
Collaborator Author

Schrodinger71 commented Jul 21, 2024

Надо всё проверить, тесты прошли

@Schrodinger71
Copy link
Collaborator Author

ну и с хера

@Schrodinger71
Copy link
Collaborator Author

Жесть

@Schrodinger71 Schrodinger71 enabled auto-merge July 26, 2024 20:47
@Schrodinger71 Schrodinger71 disabled auto-merge July 26, 2024 23:38
@Schrodinger71 Schrodinger71 merged commit 48402eb into master Jul 26, 2024
11 checks passed
@Schrodinger71 Schrodinger71 deleted the upstream2107 branch July 26, 2024 23:39
@Schrodinger71 Schrodinger71 removed the Не Мерджить Не МЕРДЖИТЬ НИ В КОЕМ СЛУЧАЕ label Jul 27, 2024
@Schrodinger71 Schrodinger71 removed their assignment Jul 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение локализации Changes: Map Изменение карт Changes: Sprites Изменение спрайтов Changes: UI Изменение интерфейса
Projects
None yet
Development

Successfully merging this pull request may close these issues.