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

Cargo Gift Event: Add chance for toy boxes, party supplies, art supplies, variety instrument crates or plushie crates #32228

Closed
wants to merge 3 commits into from

Conversation

august-sun
Copy link
Contributor

@august-sun august-sun commented Sep 17, 2024

About the PR

Added the chance for crates of toys, party supplies, art supplies, instruments or plushies to be gifted to the station!

Why / Balance

Fun gift crates for fun crews!

Technical details

Edited cargo_gifts.yml and cargo-gifts.ftl to match.

Media

Requirements

Breaking changes

Changelog

🆑

  • add: Added the chance for the cargo gift event to include crates of toys, party supplies, art supplies, instruments or plushies.

@github-actions github-actions bot added the No C# For things that don't need code. label Sep 17, 2024
…e chars to read better when game rule is added
@august-sun
Copy link
Contributor Author

Fixed the issue I was having with the exception, also added some minor character changes for gift descriptions to read better when it triggers

@august-sun august-sun marked this pull request as ready for review September 17, 2024 21:45
parent: CargoGiftsBase
components:
- type: StationEvent
weight: 3
Copy link
Member

Choose a reason for hiding this comment

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

it will make the likelihood of useful gifts falling out much less likely.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are more meant to be fun additions rather than specifically be useful; what should they be weighted instead?

@slarticodefast
Copy link
Member

Can you make your PR description and changelog a little more descriptive than "fun"? Mention what crates and the event they are gifted in.

@august-sun august-sun changed the title Fun cargo gift crates Cargo Gift Event: Add chance for toy boxes, party supplies, art supplies, variety instrument crates or plushie crates Sep 18, 2024
@august-sun
Copy link
Contributor Author

Can you make your PR description and changelog a little more descriptive than "fun"? Mention what crates and the event they are gifted in.

Done!

@slarticodefast slarticodefast added the Undergoing Maintainer Discussion This PR is currently going through an internal discussion by the maintainer team. label Sep 19, 2024
@august-sun
Copy link
Contributor Author

Closing until this is ready with more to add!

@august-sun august-sun closed this Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No C# For things that don't need code. Undergoing Maintainer Discussion This PR is currently going through an internal discussion by the maintainer team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants