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

Stardew Valley: Fixed Help wanted rules, added missing coffee bean to cropsanity #2089

Merged

Conversation

agilbert1412
Copy link
Collaborator

@agilbert1412 agilbert1412 commented Aug 7, 2023

What is this fixing or adding?

The Help Wanted setting, when using a number not divisible by 7, was missing some logic rules for the remainder. Now the rules should be set properly
The cropsanity setting was missing one crop, the coffee bean. This adds it

How was this tested?

Ran unit tests, generated a couple of worlds.

@agilbert1412 agilbert1412 changed the title Stardew Valley: Added missing coffee bean to cropsanity Stardew Valley: Fixed Help wanted rules, added missing coffee bean to cropsanity Aug 7, 2023
@Berserker66 Berserker66 merged commit 01e8e95 into ArchipelagoMW:main Aug 11, 2023
@ThePhar ThePhar added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label Aug 16, 2023
FlySniper pushed a commit to FlySniper/Archipelago that referenced this pull request Nov 14, 2023
… cropsanity (ArchipelagoMW#2089)

* - Added missing coffee bean to cropsanity

* - Fix an issue with the seed having the same name as the crop

* - Fix a recently discovered bug with help wanted rules when using a number not divisible by 7
@agilbert1412 agilbert1412 deleted the StardewValley/CropsanityCoffeeBean branch December 6, 2023 14:42
kl3cks7r pushed a commit to kl3cks7r/Archipelago that referenced this pull request Dec 15, 2023
… cropsanity (ArchipelagoMW#2089)

* - Added missing coffee bean to cropsanity

* - Fix an issue with the seed having the same name as the crop

* - Fix a recently discovered bug with help wanted rules when using a number not divisible by 7
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
… cropsanity (ArchipelagoMW#2089)

* - Added missing coffee bean to cropsanity

* - Fix an issue with the seed having the same name as the crop

* - Fix a recently discovered bug with help wanted rules when using a number not divisible by 7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants