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

Rework wizard polymorph to timed status effect #17563

Conversation

flappybatpal
Copy link
Contributor

@flappybatpal flappybatpal commented Jan 4, 2024

About the PR

Changes wizard polymorph to use a status effect that will revert you to your human form after eight minutes.
Stores the original mob in null with hibernating set so they can return with the same limbs, organs etc.

Why's this needed?

Being permanently stuck as an animal critter was kind of lame.

Changelog

(u)Flappybat
(*)Wizard polymorph now changes you into an animal for eight minutes. When the spell expires your original human form and status will be restored. Status icon by CalliopeSoups.

@flappybatpal flappybatpal added C-Balance Balance changes, buffs and nerfs C-Rework Reworks a feature E-Add-To-Wiki A PR that will require changes to the wiki A-Status-Effects Deals with status effects of any kind labels Jan 4, 2024
@github-actions github-actions bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jan 4, 2024
@flappybatpal
Copy link
Contributor Author

Removed the assigned_role changes and check because frankly I couldn't get it to work and it seemed a bit unnecessary now you transform back.

The original mob is qdel on the assumption we probably don't want a build up of humans at null if the critters die.

flappybatpal and others added 4 commits January 5, 2024 09:08
Co-authored-by: FlameArrow57 <53062374+FlameArrow57@users.noreply.github.com>
Co-authored-by: FlameArrow57 <53062374+FlameArrow57@users.noreply.github.com>
@boring-cyborg boring-cyborg bot added the C-Sprites Automatically applied on any .dmi or icons folder change label Jan 5, 2024
@pgmzeta
Copy link
Contributor

pgmzeta commented Jan 5, 2024

not seeing the new sprite when testing locally

Co-authored-by: TobleroneSwordfish <20713227+TobleroneSwordfish@users.noreply.github.com>
@github-actions github-actions bot added the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Jan 20, 2024
@flrsh
Copy link
Contributor

flrsh commented Jan 20, 2024

Imo four minutes is too short especially when the cooldown is already over two minutes. I think something like seven or eight minutes would be more impactful while not being horrible.

@github-actions github-actions bot removed the S-Merge-Conflict Applied and removed when a PR has or no longer has a merge conflict label Jan 23, 2024
Copy link
Contributor

@TobleroneSwordfish TobleroneSwordfish left a comment

Choose a reason for hiding this comment

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

lgtm

@github-actions github-actions bot added the S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) label Jan 27, 2024
@TobleroneSwordfish TobleroneSwordfish merged commit b7822b3 into goonstation:master Jan 29, 2024
24 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 29, 2024
@Studenterhue Studenterhue removed the E-Add-To-Wiki A PR that will require changes to the wiki label Feb 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Status-Effects Deals with status effects of any kind C-Balance Balance changes, buffs and nerfs C-Rework Reworks a feature C-Sprites Automatically applied on any .dmi or icons folder change S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants