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
Insert blocks into remixed playlab levels to handle different level settings #15207
Conversation
// The excluded app types need to make modifications to the project that | ||
// apply to the remixed project, but should not be saved on the original | ||
// project. See (Turtle|Studio).prepareForRemix(). | ||
return !['turtle', 'studio'].includes(appOptions.app); |
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.
What happens if I try to remix one of these apps while logged out?
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.
Ah, yeah it used to force you to log in and I broke that. Fixed by just disabling the last save.
@@ -2,6 +2,7 @@ import Collidable from './collidable'; | |||
import * as constants from './constants'; | |||
import _ from 'lodash'; | |||
import { ShakeActor } from './spriteActions'; | |||
import Studio from './studio'; |
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.
Is Studio
already global?
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.
Yes, but it seems to get set somewhere that doesn't run in these unit tests. This file is completely untested 😿
Differing map layouts lead to a bunch of generated blocks, e.g.
Before Remix:
After Remix:
There would be even more if the original level also specified custom sprites/emotions/sizes/speeds.