Skip to content

Agrega Guia #95

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

Merged
merged 12 commits into from
Mar 2, 2019
Merged

Agrega Guia #95

merged 12 commits into from
Mar 2, 2019

Conversation

gomezgleonardob
Copy link
Contributor

No description provided.

Copy link
Contributor

@rpalaciosg rpalaciosg left a comment

Choose a reason for hiding this comment

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

Comentarios y Sugerencias.


Abierto el email nos dirigimos a la bandeja de entrada y verificamos el email de la P.S.F. ,caso contrario ir a la carpeta Spam

En el mensaje nos indica::
Copy link
Contributor

Choose a reason for hiding this comment

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

Hola, veo que no ha pasado las pruebas de CI, quiero sugerir algo, estoy aprendiendo a usar github espero pueda hacerme entender. En está parte del texto línea 22 a la 31 que muestra el mensaje de la PSF que llega al correo; está bien mostrarlo en una nota o tipo caja topic, pero me parece que la sintaxis seria así.

.. topic:: En el mensaje nos indica:

    Hello from python.org!

    You're receiving this e-mail because user xxxxxx at python.org has given yours as an e-mail address to connect their account.

    To confirm this is correct, go to  `<http://www.python.org/accounts/confirm-email/xxxxxxxxxxxxxxxxxxxxxxxx>`_

    Thank you from python.org!
    python.org

O también se podría evitar dar formato de vinculo a la URL en la línea 28 para evitar este error; además al ser link que simboliza o representa el original, dentro del contexto del tutorial puede no ser tan necesario.

Espero sea de ayuda.

Copy link
Contributor

@rpalaciosg rpalaciosg Oct 26, 2018

Choose a reason for hiding this comment

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

Y disculpas, no supe como señalar el comentario en todas esas líneas, quise poner un comentario, no solicitud de cambios, espero no dañar nada.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Gracias ,por tu ayuda

Copy link
Member

Choose a reason for hiding this comment

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

@rpalaciosg muy bien el review!

no supe como señalar el comentario en todas esas líneas

Es una de las restricciones de GitHub, sólo se puede hacer un comentario sobre una linea.

Copy link
Contributor

Choose a reason for hiding this comment

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

De nada @gomezgleonardob me alegra haber ayudado.
y gracias @stsewd estar trabajando en el mini tutorial de reestructuredtext me ayudó.
No sé si es por la poca experiencia en rst, pero creo que aun prefiero markdown.

Copy link
Member

Choose a reason for hiding this comment

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

reStructured text tiene una syntaxis rara, pero es mucho más estándar y extensible que markdown

Copy link
Contributor

Choose a reason for hiding this comment

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

Debe ser tu que tienes mas experiencia, con todo voy a seguirlo probando y usando para ganar mas experiencia.

Copy link
Member

@stsewd stsewd left a comment

Choose a reason for hiding this comment

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

Además de hacer que pasen las pruebas, nos faltan imágenes y los metadatos que debe tener cada página para que sea renderizada con Nikola. Puedes buscar en las otras páginas para tomar de referencia.

Puedes ejecutar las pruebas localmente con

rstcheck -r .
nikola build --strict

También, el nombre del archivo preferiría que sea en minísculas pages/guias/psf.rst


.. topic:: Datos:

* Username : El nombre del usuario con el que deseamos registrarnos
Copy link
Member

Choose a reason for hiding this comment

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

No debe ir un espacio antes de :


En el navegador web abrimos una una nueva pestana e ingresamos al email con que nos registramos

Abierto el email nos dirigimos a la bandeja de entrada y verificamos el email de la P.S.F. ,caso contrario ir a la carpeta Spam
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Abierto el email nos dirigimos a la bandeja de entrada y verificamos el email de la P.S.F. ,caso contrario ir a la carpeta Spam
Abierto el email nos dirigimos a la bandeja de entrada y verificamos el email de la P.S.F, caso contrario ir a la carpeta Spam

@stsewd stsewd added the wip Trabajo en progreso label Nov 15, 2018
@gomezgleonardob
Copy link
Contributor Author

  • Update Files and Images
  • Please check the commit

@stsewd
Copy link
Member

stsewd commented Mar 2, 2019

Aún falta el paso de registrarse como miembro, hasta ahora sólo está el registro en la web, hay que hacer otro paso para tener la membresía de la PSF

@stsewd
Copy link
Member

stsewd commented Mar 2, 2019

Voy a hacer merge, pero debemos iterar más cambios para cerrar por completo #60

@stsewd stsewd merged commit dfb37f6 into pythonecuador:src Mar 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wip Trabajo en progreso
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants