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] Blood brother gives chat log for conversions, fix blood brothers getting more antags (maybe?), better admin logging #978

Merged
merged 1 commit into from
Dec 5, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25438
Original PR: tgstation/tgstation#80095

About

Closes #80094.

I think I fixed the issue of blood brothers getting more antags by setting special_role. Dynamic rulesets are too complicated and this shouldn't even be possible as a bug. I intend to completely rewrite how rulesets work.

Changelog

🆑 Mothblocks
qol: Converting someone will now give a chat message.
fix: Blood brothers can no longer get other antagonists, I hope.
/:cl:

…hers getting more antags (maybe?), better admin logging [MDB IGNORE] (#25438)

* Blood brother gives chat log for conversions, fix blood brothers getting more antags (maybe?), better admin logging (#80095)

I think I fixed the issue of blood brothers getting more antags by
setting special_role. Dynamic rulesets are too complicated and this
shouldn't even be possible as a bug. I intend to completely rewrite how
rulesets work.

* Blood brother gives chat log for conversions, fix blood brothers getting more antags (maybe?), better admin logging

---------

Co-authored-by: Mothblocks <[email protected]>
@Iajret Iajret merged commit 7fac73e into master Dec 5, 2023
23 checks passed
@Iajret Iajret deleted the upstream-mirror-25438 branch December 5, 2023 09:46
AnywayFarus added a commit that referenced this pull request Dec 5, 2023
Iajret pushed a commit that referenced this pull request Feb 17, 2024
* Adds support for different achievement dmis (#81471)

## About The Pull Request

I am back to pushing my code improvements upstream, sorry in advance.
As the title says, this adds support for achievement icons to be in
different dmi files than the default, pretty much the exact same way
that Language icons (for chat assets) do.

## Why It's Good For The Game

It is one of the few things in game (the only other thing i can think of
that does this rn is barsigns) that cannot have their icon changed when
trying to make a subtype, this is very limiting and very annoying,
especially since dmi conflicts are one of the worst types to deal with,
this would make my life a lot easier.

## Changelog

No player-facing changes.

* Adds support for different achievement dmis

---------

Co-authored-by: John Willard <[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