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 Almanac for 1.6.10 Without Random Features #35

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ShotrayWW
Copy link

I've commented out the Weather, Luck, and Notices modules and 'fixed' the remaining errors.

Tested with SVE, I changed some codes about fish locations. I had to replace the getLocName() function in FishHelper.cs with displayname directly. This will make all types of farms be listed in fishing locations. (I couldn't make it work with the check enabled, but if I removed the check, it worked by using displayname directly.)

The function Common_SpriteText_Patches.Patch in ModEntry.cs is removed from Commons. It seems that nothing more was added when Better Crafting was updated with this function removed, so I just uncommented another patch method.

I also changed the Chinese localization to fit the translation in the Wiki and edited for SVE 1.15's new location.

I'm not sure if this helps, but just wanted to share it.

@n1ghthouse
Copy link

hey @xysztm is it possible to know how to compile the mod or if there are any complied versions available for your fork?

@Vindeflei
Copy link
Contributor

Hi, just wanted to let you know that I merged your pull request into my own branch of KL's mods since you fixed a bug I'd been having with seasonal fish availabilities, and I'd also worked on adding in the Random stuff and the Notices page, like the Bookseller and Green Rain months ago. Since you'd just mass-commented out those pages it was really easy to merge your branch in, so thank you for that! I mentioned you in my commit when I merged the branch in and I'm going to credit you on my pull request as well, and recommend merging yours in first.

@Vindeflei
Copy link
Contributor

Also @n1ghthouse I put my build up here for temporary use: https://github.com/Vindeflei/KhloeLeclairStardewMods/releases/tag/Almanac_1_18_2

@n1ghthouse
Copy link

Also @n1ghthouse I put my build up here for temporary use: https://github.com/Vindeflei/KhloeLeclairStardewMods/releases/tag/Almanac_1_18_2

thank you sm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants