Skip to content

Puts the Fist of Sol back in the bag.#2017

Merged
Bokkiewokkie merged 9 commits intoBeeStation:masterfrom
QAI-07:fist-deleteus
Oct 3, 2022
Merged

Puts the Fist of Sol back in the bag.#2017
Bokkiewokkie merged 9 commits intoBeeStation:masterfrom
QAI-07:fist-deleteus

Conversation

@QAI-07
Copy link
Copy Markdown
Contributor

@QAI-07 QAI-07 commented Aug 14, 2022

About The Pull Request

For a while now, on rare occasions, the Fist of Sol would appear in the Rubicon sector along with an identical clone of itself. While it still works as intended, it's appearance outside of strict admin intervention was not intended. I've just finally gotten up to fix it.

Why It's Good For The Game

Fix man good.
Also, I can now buff it without worry of it appearing randomly and jumpscaring the crew with funky new weapons.

Changelog

🆑
fix: fixed the Fist of Sol Boss showing up when it shouldn't.
/:cl:

QAI-07 added 2 commits August 14, 2022 22:14
This should, in theory, stop it from spawning in places that just ask for any syndicate vessel.
It should still be spannable by admins while not appearing under any natural circumstances, as it was intended from the start.
Copy link
Copy Markdown
Contributor

@Bokkiewokkie Bokkiewokkie left a comment

Choose a reason for hiding this comment

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

just a small thing...

Comment thread nsv13/code/modules/overmap/ai-skynet.dm Outdated
@Bokkiewokkie Bokkiewokkie added Fix Overmap This issue or PR is related to overmap interactions labels Aug 16, 2022
Copy link
Copy Markdown
Contributor

@DeltaFire15 DeltaFire15 left a comment

Choose a reason for hiding this comment

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

Instead of changing its type away from ../syndicate/ai which leads to issues with things that check for this subtype, add it to the enemy_blacklist in ../subsystem/starsystem.dm.

@red031000
Copy link
Copy Markdown
Contributor

image
doesn't work

@red031000
Copy link
Copy Markdown
Contributor

ok, correction, does work, however the same issue happens to sol's revenge, might be an idea to fix it as well?

@Pontenerd
Copy link
Copy Markdown
Contributor

image
round id is 4800

@Bokkiewokkie
Copy link
Copy Markdown
Contributor

Bokkiewokkie commented Aug 24, 2022

round id is 4800

SyndicateFlagshipInfograph

@Bokkiewokkie
Copy link
Copy Markdown
Contributor

@QAI-07 Are you going to adress DeltaFire's requested changes?

@QAI-07
Copy link
Copy Markdown
Contributor Author

QAI-07 commented Sep 17, 2022

@Bokkiewokkie I would if I had even the slightest idea of how to do any of that.

So i'm just gonna start guessing until the code complies and assume it works.

@QAI-07 QAI-07 requested a review from DeltaFire15 September 18, 2022 18:48
Comment thread nsv13/code/modules/overmap/types/syndicate.dm Outdated
Remove Kebab:

Commits suggestions by Bokkiewokkie

Co-authored-by: Bokkiewokkie <43698041+Bokkiewokkie@users.noreply.github.com>
@Bokkiewokkie Bokkiewokkie merged commit 7685f63 into BeeStation:master Oct 3, 2022
@QAI-07 QAI-07 deleted the fist-deleteus branch November 17, 2023 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix Overmap This issue or PR is related to overmap interactions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants