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

Next challenge buttons #878

Merged
merged 27 commits into from
Mar 25, 2022
Merged

Next challenge buttons #878

merged 27 commits into from
Mar 25, 2022

Conversation

asanzo
Copy link
Contributor

@asanzo asanzo commented Dec 10, 2021

Selección_999(838)

  • Agrega botones de siguiente ejercicio.
  • Refactor muy grande
  • Falta testear que aparezcan / desaparezcan los botones cuando corresponde
  • Falta hacer el componente para que el header no quede tan spaghetti

Fixes #576
Fixes #320

@asanzo asanzo requested a review from a team as a code owner December 20, 2021 18:24
@asanzo
Copy link
Contributor Author

asanzo commented Jan 10, 2022

Bueno, gracias al grafiquito de ejecución de @tfloxolodeiro, pude proponer una solución a los flaky tests:

javascriptCode

Básicamente es en lugar de usar el javascriptCode de EmberBlockly desde pilasBlockly.js, usar directamente Blockly.workspaceToCode. EmberBlockly aparentemente no está actualizando javascriptCode suficientemente rápido.

Ahora metí otros problemas, pero tengo que rollbackear algunos async que metí por allí y creo que se arregla. Sigo gastándole un poquito de tiempo a esto.

@PalumboN
Copy link
Contributor

PalumboN commented Jan 10, 2022

Chetardo ese gráfico! Debe estar en una wiki o algo así. Recuerdo que tenía que hacer todo el recorrido cada vez que quería tocar algo y claramente me olvidaba al otro día 👏
La radio está muy buena.

@asanzo
Copy link
Contributor Author

asanzo commented Jan 10, 2022

Copy link
Contributor

@tfloxolodeiro tfloxolodeiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Genial 🚀. No tengo casi nada para comentar específicamente, me parece todo coherente con lo que habíamos hablando en su momento.

Aprecio la purga de parámetros y métodos innecesarios.

@asanzo asanzo merged commit efa5460 into develop Mar 25, 2022
@asanzo asanzo deleted the next-challenge branch March 25, 2022 05:31
@asanzo asanzo mentioned this pull request Apr 7, 2022
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.

Botón Siguiente Ejercicio No responde correctamente ante cambios en la URL
3 participants