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

Update for LootGames and TooMuchLoot Chest3 #18156

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

EagleEyeLogic
Copy link

Actual config file for LootGames is a very old one and correspond not with the new created config, so delete this old file.

Also provide an edited rewards.cfg because TooMuchLoot provide only the content for the chests 1-4 but do not overwrite the rolls/picks per chest correctly.

With the default rewards.cfg we get

4-8 rolls/picks for chest 1 (wanted only 2-4)
5-10 rolls/picks for chest 2 (wanted only 3-6)
6-11 rolls/picks for chest 3 (wanted only 4-8)
7-12 rolls/picks for chest 4 (wanted only 5-10)

Update chest 3 as Forestry Farm Blocks are

:0 for Farm Block
:2 for Gear Box
:3 for Hatch
:4 for Valve
:5 for Controller

and goal should be to have a chance for an early MultiFarm. So most FarmBlocks, than hatch, valve, gear and lowest amount for controller.

@Dream-Master Dream-Master requested a review from a team November 28, 2024 20:55
@chochem chochem added ongoing freeze - don't merge PR label; Not just a bug fix and thus affected by a current freeze for an upcoming version Affects Balance Change affecting balance. This PR requires admin approval before merging. labels Nov 28, 2024
@EagleEyeLogic
Copy link
Author

Only to be clear for that, you say from chest1.xml to chest4.xml how much picks you like to have at min and max.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Affects Balance Change affecting balance. This PR requires admin approval before merging. ongoing freeze - don't merge PR label; Not just a bug fix and thus affected by a current freeze for an upcoming version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants