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

Migrated to SMAPI 4.0.0 and Stardew Valley 1.6 #30

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

Conversation

chsiao58
Copy link

@chsiao58 chsiao58 commented Feb 7, 2024

BetterArtisanGoodIcons has a warning message for using deprecated SMAPI method that will be removed in SMAPI 4.0.0, this update fixed that.

The package also doesn't build with .NET 4.5.2, so it was updated to build with .NET 5.0 instead as suggested by Stardew Valley Modding guideline.

@akauangr
Copy link

Can you provide a release in your forked project?

@chsiao58
Copy link
Author

chsiao58 commented Apr 1, 2024

I technically COULD, but there would be no point to do that, because although my current fork fixed SMAPI 4.0 warning massage, this version is still only for Stardew Valley 1.5, it doesn't support Stardew Valley 1.6 just yet, and I don't really see a point providing release for previous Stardew Valley version.

I am still in the middle of updating this mod for 1.6 so check back later.

@chsiao58 chsiao58 changed the title Migrated to SMAPI 4.0.0 by updating deprecated code Migrated to SMAPI 4.0.0 and Stardew Valley 1.6 by updating deprecated code Apr 4, 2024
@chsiao58 chsiao58 changed the title Migrated to SMAPI 4.0.0 and Stardew Valley 1.6 by updating deprecated code Migrated to SMAPI 4.0.0 and Stardew Valley 1.6 Apr 4, 2024
@chsiao58
Copy link
Author

chsiao58 commented Apr 11, 2024

I have unofficial fix for BAGI on my repo, you can download it here if you need it.

As for pull request to this original mod, I will leave it open just in case.

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