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

feat(balance): merge and rebalance the two liquid fertilizer recipes #5510

Merged
merged 1 commit into from
Oct 5, 2024

Conversation

chaosvolt
Copy link
Member

Checklist

Required

Optional

Purpose of change

So I looked at the recipe for liquid fertilizer and noticed two recipes that seemed near-identical except in a few narrowly-specific annoying ways, swapping out a single identically-interchangeable component but making the tool requirements radically different.

Describe the solution

  1. Merged the two recipes for liquid fertilizer into one, obsoleting the "simple" version. This means aluminum sulfate added as an alternative to the ammonia/lye/etc component, using the skill demands and book options of the harder version, but also compromising on what tool quality to use by allowing CHEM 1 instead of using BOIL 1 or CHEM 2.
  2. Moved the unneeded other recipe to the obsoletion folder.

Describe alternatives you've considered

screm

Testing

  1. Checked affected files for syntax and lint errors.
  2. Ported changes to playthrough release and load-tested.

Additional context

@github-actions github-actions bot added the JSON related to game datas in JSON format. label Oct 5, 2024
@chaosvolt chaosvolt merged commit afd8da4 into cataclysmbnteam:main Oct 5, 2024
9 checks passed
@chaosvolt chaosvolt deleted the fertilizate-me-uwu branch October 5, 2024 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JSON related to game datas in JSON format.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants