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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Converts deployables into a component #9747

Merged
merged 1 commit into from
Sep 4, 2023

Conversation

PowerfulBacon
Copy link
Member

About The Pull Request

  • Converts deployable items into a component.
  • Makes it so dragging an open bodybag onto you will try to zip it up automatically.
  • Fixes the bot item not being properly initialised on pickup

Why It's Good For The Game

Makes deployables more versatile code-wise as they no longer have to be typed to /obj/item/deployable.

Testing Photographs and Procedure

deployables.mp4
deployables.2.mp4
deployables.3.mp4

Changelog

馃啈
code: Deployables are now a component.
tweak: Dragging a bodybag onto your mob while it is open will try to zip it up instead of instantly failing.
/:cl:

Deployable component

Fixes duplication glitch, fixes shield sat icon

Update deployable.dm

Update bot.dm
Copy link
Contributor

@PestoVerde322 PestoVerde322 left a comment

Choose a reason for hiding this comment

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

The map changes are just object renaming, approved.

Copy link
Contributor

@Rukofamicom Rukofamicom left a comment

Choose a reason for hiding this comment

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

Looks good to me, sorry this took so long

@Rukofamicom Rukofamicom added this pull request to the merge queue Sep 4, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 4, 2023
@Rukofamicom Rukofamicom added this pull request to the merge queue Sep 4, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 4, 2023
@Rukofamicom Rukofamicom added this pull request to the merge queue Sep 4, 2023
Merged via the queue into BeeStation:master with commit 6671223 Sep 4, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants