-
Notifications
You must be signed in to change notification settings - Fork 45
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
Patch Lighting.Clocktime
to serialize
#315
Conversation
I'm a little lost here - did |
Sorry, I was in a hurry when I made this PR and didn't provide as much background as I should have. Rojo has gotten intermittent reports of Another solution would be to provide a migration of some sort for |
Not sure about this one, tbh. I could totally see someone deserializing a place using Lune, ( |
In what can only be described as cursed, Roblox seems to load whichever property appears first in the file. This means Given this, I'm not sure what to do. A migration would probably work, though I'm not sure how Rojo would handle a file that had both if migrations were involved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's fine to serialize both until #277 is more fleshed out... the possible inconsistency kinda sucks, but hopefully it won't be too surprising to users
Like it says on the tin.