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

[MIRROR] Fix medibots #987

Merged
merged 1 commit into from
Dec 6, 2023
Merged

[MIRROR] Fix medibots #987

merged 1 commit into from
Dec 6, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25454
Original PR: tgstation/tgstation#80136

About The Pull Request

Should fix advanced medkit medibots healing and also give back the healing bonus to brute medkit medibots. Medibots should no longer leave their healing pose in the middle of healing either. Player medibots can't overheal.

It also readds any removed lines. And they can use robot emotes again.
Fixes #80135

Why It's Good For The Game

Fixes!

Changelog

🆑 Vermidia
fix: Medibots made from advanced medkits works again
fix: Medibots made from brute medkits have their bonus healing again
fix: Medibots can use robotic emotes again
/:cl:

* Fix medibots

* Conflict!

---------

Co-authored-by: Verm <[email protected]>
Co-authored-by: SomeRandomOwl <[email protected]>
@ReezeBL ReezeBL merged commit 26e389b into master Dec 6, 2023
23 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-25454 branch December 6, 2023 08:16
AnywayFarus added a commit that referenced this pull request Dec 6, 2023
Iajret pushed a commit that referenced this pull request Feb 17, 2024
…, not inspired by any relic at all (#987)

* Adds a new mobility option for wizards, the Telegram Scepter, not inspired by any relic at all (#81413)

## About The Pull Request

This PR adds a new mobility option for wizards, the Telegram Scepter, a
magical rod that teleports you anywhere you point it.

But be warned, its method of teleportation is very volatile, putting you
in a state of flux. While the scepter itself has no cooldown, using it
while in flux will damage you severely and purge some of your chems. In
fact, any form of magical teleportation will damage you while in flux.


https://github.com/tgstation/tgstation/assets/51863163/b1008f6c-d724-4b7d-8e47-b70e2e770183

Non-wizards who attempt to utilize the wand will find themselves in a
state of permanent flux, meaning any and all teleports are met with
damage. It's also less precise.

Also adds an admin-only version which has infinite range and applies no
flux.

Any semblances to any other game is purely coincidental, I assure you.

Misc. Details: 

- I added the ability to pass sound datums to playsound, so it doesn't
override custom sound settings you may want.
- I moved the teleport signals out of the pure-sounding proc `check
teleport`.

## Why It's Good For The Game

I've always thought wizard mobility has been rather lacking, in that you
generally only pick Jaunt.


![image](https://github.com/tgstation/tgstation/assets/51863163/6eb2b136-7ec4-445e-b28d-c8841e5bcbe9)

So when someone threw an offhanded suggestion to make a certain item
from a certain game in discord, my wheel started to turn.

And here we are.

- Teleporting anywhere you can see is very powerful (especially in
conjunction with the Scrying Orb) so this makes it a very high potency
item a wizard can pick up, almost like a targeted blink.
- It is, however, offset by a few things: Namely, the flux mechanic
putting a soft cooldown on teleporting (lest you kill yourself), and it
being an item, meaning you may have to pick up Instant Summons for added
safety.

Overall, I thought it to be a very flavorful mobility option for
wizards. Maybe it will shake up spell selection. Who knows?

## Changelog

:cl: Melbert
add: Wizards have a new mobility option available, the Telegram Scepter.
The ability to travel anywhere you can see at the point of a wand... but
at a price?
/:cl:

* Adds a new mobility option for wizards, the Telegram Scepter, not inspired by any relic at all

---------

Co-authored-by: MrMelbert <[email protected]>
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.

3 participants