-
Notifications
You must be signed in to change notification settings - Fork 5
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
Botón ejecutar - parte 1 #302
Conversation
"left": "Izquierda", | ||
"left": "la izquierda", |
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.
Cambiamos estas traducciones como las que estan hoy en producción (minúscula y con el "la" antets)
@@ -58,7 +57,7 @@ const createGenericJSCode = (id: string, customCode: string) => { | |||
if (variable_name === "DO") { | |||
variable_object = generator.statementToCode(block, variable_name); | |||
} else { | |||
variable_object = generator.valueToCode(block, variable_name) || block.getFieldValue(variable_name) || null; | |||
variable_object = generator.valueToCode(block, variable_name, Order.ATOMIC) || block.getFieldValue(variable_name) || null; |
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.
Sin esto no andan los parámetros en los bloques.
declare module 'js-interpreter' { | ||
export default class Interpreter { | ||
createNativeFunction: (func: any) => void | ||
createAsyncFunction: (func: any) => void | ||
constructor(code: string, initFunc?: (interpreter: Interpreter, globalObject: any) => void) | ||
step(): boolean | ||
run(): boolean | ||
appendCode(code: string): void | ||
paused: boolean | ||
setProperty(obj: any, name: string, value: any, descriptor?: PropertyDescriptor): void; | ||
} | ||
} |
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.
No tenía tipos js-interpreter
|
||
return <Stack direction="row" flexWrap={"wrap"} flexGrow={1}> | ||
{blocklyWorkspace} | ||
<Stack> | ||
<SceneButtons /> | ||
<SceneButtons challenge={challenge} /> |
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.
Al final terminé pasando el challenge por parámetro, creo que el tema del context lo puedo hacer en otro PR para no extender este issue.
WOOOOOOOOOO9OIIII GENIAL Sabés qué es bueno para todo refactor??? LOS TESTS Ahora quiero una tarde de pilasbloquear para sacar esos tests. |
Toda tuya !!!!! |
If you want... we are prepared for one of your barbecues (I can take care of the costs, no problem because I haven't a "parrilla" to cook it) and we can discuss about the tests... hahahhaahah!!!! |
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.
Hermoso!!! Bajado y probado !!!
💯
🚀
Related #251
Funciona tanto vertical como horizontal. No andan los procedimientos ni la repetición simple (falta ver qué pasa en el código que escupen), y faltan tests que van para otro PR.
Grabacion.de.pantalla.desde.24-07-24.17_29_54.mp4
Grabacion.de.pantalla.desde.24-07-24.17_34_50.mp4