search

Sketchy502 / SDV-Summary

An application to display a summary of the player from a Stardew Valley save file.
206 stars 21 forks source link

Unable to upload farms with beta cabins on them #35

Closed ken5577 closed 5 years ago

ken5577 commented 6 years ago

The uploader will not accept uploads with the cabins on them. Understandable, as it's from an update, but I was hoping to use the uploader to show my buddies a half decent layout with the cabins on the map too.

Laukei commented 6 years ago

In the 1.3 beta the save format changed structure substantially to support multiple players, so we're having to rewrite a lot of stuff at the back end of the website remembering that we have to still support older versions. We're working in branch https://github.com/Sketchy502/SDV-Summary/tree/v1.3 on this. We hope to get support rolled out as soon as our changes are stable!

Laukei commented 6 years ago

This should be fixed now, please let me know!

ken5577 commented 6 years ago

It now works for saves generated post 1.3 update, but will not accept a save from pre-1.3 with cabins on them it seems.

Laukei commented 6 years ago

You have a savegame from pre-1.3 with cabins? Can you send it me? My email's on my GitHub profile.

ken5577 commented 6 years ago

Alright, I sent over the save file for you.

Laukei commented 6 years ago

Hmm... so, the issue is that your savegame has a Cabin that doesn't have a farmhand inside it. All the savegames we've managed to make in v1.3 have farmhand inside Cabin. How did you add your cabins to your game? Also, how did you get them pre-v1.3?

ken5577 commented 6 years ago

These cabins were built by Robin, with the intent of using the farm layout as a reference for me and my buddies to use for our coop farm. By "pre-1.3," I mean the save was created before 1.3, but I am currently running 1.3, sorry for the confusion on that.

Laukei commented 6 years ago

Oh, interesting. I'll have a look and see if I can replicate this myself. In the mean time, I pushed a fix, but without a farmhand to check the cabin level, it will only show cabins as not upgraded. I can't see any information about cabin upgrade level without a farmhand, so maybe it's not possible to upgrade them unless they're owned? I don't know... anyway, should be fixed! Let me know!

ken5577 commented 6 years ago

Yup, I can now upload the farm. As for the cabin upgrades, I have yet to see any way for someone not using the cabin to upgrade it, iirc.