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

Paramedic Lockers, Portafibs, and Chemvend Tweaks #53

Merged
merged 12 commits into from
Oct 5, 2023

Conversation

rosieposieeee
Copy link
Contributor

@rosieposieeee rosieposieeee commented Sep 28, 2023

About the PR

Added the content from my previous PRs on the original branch. As a recap of these PRs:

  • Portafibs are smaller, portable defibs with less charge, for Paramedics to carry around to help people who JUST died. They hardly have enough charge to really bring back anyone who has been dead for a while, making them situational.
  • Paramedic access was actually added, which means the paramedic secure windoor now works. Oh, and doctors can't nab the really valuable Paramedic items like the hud and suit and make paramedics into shittier doctors with no chem access.
  • Added more of each jug to the chemvend so that restocks will be happening less constantly.

On top of that, I made additional tweaks and fixes to those previous PRs.

  • Portafibs were only in duffels by accident, fixed that
  • added the emagged chem dispenser chemicals to the chemvend since the dispenser will not be around very often
  • gave paramed access for doctors when there's a skeleton crew
  • I also made the rollerbed inside paramedic gear the emergency one since it seems more fitting, stands out more, is less ugly, and is otherwise literally identical.
portafib.mp4

image

image

Tested everything again and it all works.

Technical details

I had to add napalm and ultravasculine bottles to our own code.

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

upped jug amounts in each vend to accomodate for their use, and adds all emagged chemicals from the dispenser to them
@github-actions github-actions bot added Changes: Sprites Changes: YML Changes any yml files labels Sep 28, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Sep 28, 2023

RSI Diff Bot; head commit 4dff633 merging into 6018cb6
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/DeltaV/Objects/Medical/portafib.rsi

State Old New Status
icon Added
inhand-left Added
inhand-right Added
ready Added
screen Added

Edit: diff updated after 4dff633

Copy link
Contributor

@FluffiestFloof FluffiestFloof left a comment

Choose a reason for hiding this comment

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

I love comments.

They are helpful in figuring what is from especially when touching the base game files

@github-actions github-actions bot added the Status: Merge Conflict Fix your PR! label Sep 28, 2023
@github-actions
Copy link
Contributor

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

Copy link
Contributor

@DebugOk DebugOk left a comment

Choose a reason for hiding this comment

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

Paramedic is not defined within Content.Shared/Access/Components/IdCardConsoleComponent.cs meaning its an unobtainable access level outside of spawning with it.

@github-actions github-actions bot removed the Changes: YML Changes any yml files label Sep 28, 2023
@rosieposieeee
Copy link
Contributor Author

Paramedic is not defined within Content.Shared/Access/Components/IdCardConsoleComponent.cs meaning its an unobtainable access level outside of spawning with it.

image
Done, whoops. Never even been hop so I didn't think about it. Will fix merge conflicts in a sec.

Copy link
Contributor

@DebugOk DebugOk left a comment

Choose a reason for hiding this comment

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

Test failures are valid. Chemvend changes introduce free money for cargo.

  Failed NoCargoOrderArbitrage [1 s]
  Error Message:
     Found arbitrage on CrateVendingMachineRestockChemVend cargo product! Cost is 3820 but sell is 6770!
  Expected: less than or equal to 3820
  But was:  6770.0d```

removed unnecessary comments (ones indicating a Delta V change while already in the Delta V directory)
Colin-Tel
Colin-Tel previously approved these changes Sep 30, 2023
Copy link
Contributor

@Colin-Tel Colin-Tel 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! We'll just have to see how this affects the balance of things.

@Adrian16199
Copy link
Contributor

Not to be chuggin logs with a meaningless comment but..... I absolutely love this PR. Atleast as a paramedic enjoyer. Being able to carry defibs without it usin half your bag at the cost of power, definitely 10/10 because consequences if you dont pay attention to it as balance, givin paramedics additional access durin skeleton crew, absolutely 10/10 because i want to be able to paramed but also get chems if im alone at medical without CMO.

@github-actions github-actions bot added the Status: Merge Conflict Fix your PR! label Oct 3, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

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

Signed-off-by: Colin-Tel <113523727+Colin-Tel@users.noreply.github.com>
Colin-Tel
Colin-Tel previously approved these changes Oct 3, 2023
Copy link
Contributor

@Colin-Tel Colin-Tel left a comment

Choose a reason for hiding this comment

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

me saying the same thing post-merge fix
Looks good! We'll just have to see how this affects the balance of things.

@DebugOk
Copy link
Contributor

DebugOk commented Oct 5, 2023

Fuck it

@DebugOk
Copy link
Contributor

DebugOk commented Oct 5, 2023

The fuck you mean merging failed

@DebugOk DebugOk merged commit 8940aec into DeltaV-Station:master Oct 5, 2023
12 checks passed
DebugOk referenced this pull request in DebugOk/Delta-v Oct 8, 2023
* add paramedic-only access

* edit chemvend

upped jug amounts in each vend to accomodate for their use, and adds all emagged chemicals from the dispenser to them

* add portafib

* added comments and tweaked rollerbed

* one line comments

* Update IdCardConsoleComponent.cs

* increase crate price

* Comment, misspelling adjustments

removed unnecessary comments (ones indicating a Delta V change while already in the Delta V directory)

* not all jugs are created equal

this wont work btw

* chemvend restock price

---------

Signed-off-by: Colin-Tel <113523727+Colin-Tel@users.noreply.github.com>
Co-authored-by: Colin-Tel <113523727+Colin-Tel@users.noreply.github.com>
@rosieposieeee rosieposieeee deleted the medical branch February 7, 2024 03:20
TypoChel1 pushed a commit to TypoChel1/Ifi07 that referenced this pull request Jul 14, 2024
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.

5 participants