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

Usar el titulo del desafio importado #1428

Merged
merged 3 commits into from
Sep 1, 2023
Merged

Conversation

danielferro69
Copy link
Contributor

Resolves #1422

Queria tocar el codigo Ember que no tengo muy aprendido realmente. No se si es la solucion correcta. Se que funciona, pero queria probar.

titulo-desafio-importado.mp4

si el desafio importado no tiene titulo pone el texto fijo 'sin titulo' cuando se pulsa el boton guardar

@danielferro69 danielferro69 requested a review from a team as a code owner August 31, 2023 15:13
Copy link
Contributor

@asanzo asanzo left a comment

Choose a reason for hiding this comment

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

¡Genial, Dani!

Marqué una boludez no importante. Yo me lo imaginaba más bien en react este issue (y mientras escribía el issue me dí cuenta de que necesitaba escribir esa idea pero no llegué por cansancio).

Si en emberCommunication >> importChallenge , donde se hace el mapeo del modelo de react al de ember, se llena el nombre de la escena, ya está.

Y de hecho eso tiene la ventaja de que todas las transformaciones están hechas en un sólo lugar, y ember no tiene lógica de React en el guardarSolucion, sino sólo en el momento de cargar un desafío.
(Sin contar que deberíamos elegir bien las modificaciones que hacemos sobre Ember, porque todas eventualmente van a volar)

Pero son detalles, podemos mergear esto.

Co-authored-by: asanzo <alfredo.sanzo@gmail.com>
@danielferro69 danielferro69 merged commit 10f0e46 into develop Sep 1, 2023
@danielferro69 danielferro69 deleted the nombrandoConElImportado branch September 1, 2023 16:33
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.

Descargar la solución de un desafío produce undefined en el nombre de archivo
2 participants