-
Notifications
You must be signed in to change notification settings - Fork 461
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
Add nested learning objective headers #1693
Conversation
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.
¡Gracias @unjust ! 😊
Perdón la demora, es harto código que revisar, y me toca ir poniéndole algunos console.log
para poder irlo entendiendo bien 🙈
Tengo sólo una duda, veo que en el comentario que dejaste hay un link hacia el README de un proyecto (md-links
), en ese README se ve de esta forma:
Pero al ejecutar el comando en mi local con los cambios, se despliega esto:
ES:
PT:
Me imagino que la intención es que, en vez de que diga js/data-types
o js/modules
, contenga su correcta traducción en cada lenguaje, será que falta algo en el PR para que se despliegue como el README que compartes? avísame! En cualquier caso, se ve bien y creo que es un cambio necesario, asegurándonos que funciona y se comporta como debe lo aprobamos nomas! 🙋♂️
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.
LGTM ⭐
Dejé sólo un comentario a modo de sugerencia en una función en la cual había una variable q no se estaba utilizando, siéntete libre de agregarla o no, no cambia el output, y los tests siguen pasando.
Mil, mil gracias @unjust por el trabajo y por haberle agregado tests 🤗 🥳
Co-authored-by: Alfredo González <12631491+mfdebian@users.noreply.github.com>
…_nested_headers_1650
…culum into add_oas_nested_headers_1650
Thanks @mfdebian will self approve and merge this after the UX release. |
Addresses #1650
Need to try again because it looks like github strips styles in markdownhttps://github.com/Laboratoria/OA-TEST-fleet-management-api/blob/main/README.md
This one may be better, adds text to headings deeper in the hierarchy
https://github.com/Laboratoria/OA-TEST2-fleet-management-api/blob/main/README.md
Example with md links
https://github.com/Laboratoria/OA-TEST-md-links