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

Various features/tweaks from ScavStation. #4807

Merged
merged 11 commits into from
Jan 31, 2025

Conversation

MistakeNot4892
Copy link
Contributor

@MistakeNot4892 MistakeNot4892 commented Jan 25, 2025

Dev-suitable tweaks/fixes from downstream. Other items have been split into their own PRs.

@MistakeNot4892 MistakeNot4892 added do not merge This PR is on hold, do not merge it. work in progress This PR is under development and shouldn't be merged. labels Jan 25, 2025
@MistakeNot4892 MistakeNot4892 force-pushed the scavs branch 11 times, most recently from 2a9f004 to 65e7d23 Compare January 25, 2025 10:51
@MistakeNot4892 MistakeNot4892 changed the base branch from staging to dev January 25, 2025 11:12
@MistakeNot4892 MistakeNot4892 force-pushed the scavs branch 2 times, most recently from 8f680dc to f12f153 Compare January 25, 2025 11:15
@MistakeNot4892 MistakeNot4892 changed the title Syncing Nebula with ScavStation's non-scav changes. Various fixes/tweaks from ScavStation. Jan 25, 2025
@MistakeNot4892 MistakeNot4892 removed the work in progress This PR is under development and shouldn't be merged. label Jan 25, 2025
@MistakeNot4892 MistakeNot4892 changed the base branch from dev to staging January 25, 2025 11:38
@MistakeNot4892 MistakeNot4892 changed the base branch from staging to dev January 25, 2025 11:39
@MistakeNot4892 MistakeNot4892 changed the title Various fixes/tweaks from ScavStation. Various features/tweaks from ScavStation. Jan 25, 2025
@MistakeNot4892 MistakeNot4892 added ready for review This PR is ready for review and merge. and removed do not merge This PR is on hold, do not merge it. labels Jan 25, 2025
@MistakeNot4892 MistakeNot4892 marked this pull request as ready for review January 27, 2025 02:09
Copy link
Member

@out-of-phaze out-of-phaze left a comment

Choose a reason for hiding this comment

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

couple nitpicks. also if you wanna add more drinks maybe take a peek at https://github.com/OpaqueSunset/LighthouseRelay/tree/master/mods/content/extradrinks


/obj/item/box/animal_cubes/spiders/WillContain()
return list(/obj/item/food/animal_cube/wrapped/spider = 5)

/obj/item/box/animal_cubes/carp
name = "space carp cube box"
desc = "Drymate brand carp cubes. Allergen information: May contain pike"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
desc = "Drymate brand carp cubes. Allergen information: May contain pike"
desc = "Drymate brand carp cubes. Allergen information: May contain pike."

i also think it'd be funny if it said something like "may contain traces of pike" or whatever. something that really understates it lol. comedy!

lore_text = "A powder used to flavor food and drinks. Unpleasant to eat a full spoonful of."
taste_description = "cinnamon"
color = "#a34b0d"
value = 0.1 //is this the monetary value? if so, should be increased a bit, cinnamon is pretty expensive
Copy link
Member

Choose a reason for hiding this comment

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

yes it is the monetary value

. = "soy London Fog"
else
. = "Baron Grey"
. = "Earl Grey"
Copy link
Member

Choose a reason for hiding this comment

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

wasn't this legally distinct for a reason? doesn't seem to be a trademark so idk

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure either.

@@ -36,7 +36,7 @@
add_to_reagents(/decl/material/solid/organic/meat, 10)

/obj/item/food/animal_cube/get_single_monetary_worth()
. = (spawn_type ? round(atom_info_repository.get_combined_worth_for(spawn_type) * 1.25) : 5)
. = (spawn_type ? round(atom_info_repository.get_combined_worth_for((islist(spawn_type) ? spawn_type[1] : spawn_type)) * 1.25) : 5)
Copy link
Member

Choose a reason for hiding this comment

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

uh, what the heck is this for. it looks like they maybe wanted to do a pickweight thing but it always just takes the first item? and it never decrements either so it's not a counter.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Per Discord, it's taking the first spawn type for things like carp cubes that have multiple possible products.

Copy link
Member

Choose a reason for hiding this comment

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

i don't see the other ones ever being used though? i only saw [1] and never pickweight

@out-of-phaze out-of-phaze added awaiting author This PR is awaiting action from the author before it can be merged. and removed ready for review This PR is ready for review and merge. labels Jan 29, 2025
Copy link
Member

@out-of-phaze out-of-phaze left a comment

Choose a reason for hiding this comment

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

uhh forgot to actually mark those as requested changes my bad

out-of-phaze
out-of-phaze previously approved these changes Jan 30, 2025
@out-of-phaze out-of-phaze added ready for review This PR is ready for review and merge. and removed awaiting author This PR is awaiting action from the author before it can be merged. labels Jan 30, 2025
Copy link
Member

@out-of-phaze out-of-phaze left a comment

Choose a reason for hiding this comment

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

Conflict resolution, prior review still stands

@out-of-phaze out-of-phaze merged commit 1775c63 into NebulaSS13:dev Jan 31, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review This PR is ready for review and merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants