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

Reworks Janitor job rewards to require a sacrificial item #2890

Merged
merged 1 commit into from Dec 1, 2020

Conversation

Carbadox
Copy link
Contributor

About the PR

Reworks the orange mop and red bucket Janitor Job Rewards to require a regular mop and bucket respectively when claiming the rewards.

Why's this needed?

Because just making items pop out of thing air is a bit silly in this case. Also Im suprised I didn't get told off for this earlier.

Changelog

(u)Carbadox
(*)Adjusted the code for Janitor job rewards, red bucket and orange mop, to require a regular mop and bucket respectively when claiming the rewards, instead of them just popping out from thin air.

@Carbadox Carbadox added the C-Code-Quality Cleans up code, refactors things to be more readable or intuitive label Nov 30, 2020
@@ -114,13 +114,21 @@ mob/verb/checkrewards()
desc = "A bucket! And it's red! Wow."
required_levels = list("Janitor"=5)
claimable = 1
claimPerRound = 1
Copy link
Member

Choose a reason for hiding this comment

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

why remove the per round - for both

Copy link
Contributor

Choose a reason for hiding this comment

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

why remove the per round - for both

I assume because "making a bucket/mop out of thin air should be limited per round, whereas making your bucket/mop look different isn't really abusable".

@ZeWaka ZeWaka merged commit 50bd059 into goonstation:master Dec 1, 2020
github-actions bot pushed a commit that referenced this pull request Dec 1, 2020
@Carbadox Carbadox deleted the janitor-rewards-proper branch December 10, 2020 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Code-Quality Cleans up code, refactors things to be more readable or intuitive
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants