programminghistorian / ph-submissions

The repository and website hosting the peer review process for new Programming Historian lessons
http://programminghistorian.github.io/ph-submissions
135 stars 110 forks source link

Propuesta de lección: Rascado web con Selenium: menús, barras de búsquedas y pop-ups JS #567

Open jenniferisasi opened 12 months ago

jenniferisasi commented 12 months ago

The Programming historian ha recibido una propuesta de lección con el título provisional "Rascado web con Selenium: menús, barras de búsquedas y pop-ups JS" por @hernandezj1 . Los objetivos de la lección son:

A fin de promover una publicación a tiempo, se ha acordado que la lección se entregará hacia finales de septiembre de 2023. El autor o autores contactará con antelación con el editor si no puede cumplir con la fecha de entrega y necesita una ampliación.

Si la lección no es entregada en la fecha acordada, el editor intentará contactar con el autor o autores de la lección. Si no recibe noticias, el ticket se cerrará. Éste podrá abrirse en el futuro a petición del autor o autores.

El principal contacto para esta lección es, por ahora, @jenniferisasi. Si se produce algún problema, el autor puede contactar con nuestra ’ombudsperson' (Silvia Gutiérrez de la Torre - http://programminghistorian.org/es/equipo-de-proyecto).

anisa-hawes commented 7 months ago

Hola @jenniferisasi,

Thank you for sharing the lesson materials. Charlotte and I will process and upload these next week. We'll add a comment here to provide you with links to the key files + a preview of the submission.

Very best, Anisa

charlottejmc commented 7 months ago

Hola @jenniferisasi y @hernandezj1 ,

Puedes encontrar los archivos aquí:

Puedes revisar la vista previa de la lección aquí:

jenniferisasi commented 7 months ago

Hola @hernandezj1:

He leído atentamente la lección y vas por buen camino, ¡felicidades! Me gusta el ejemplo, el paso por paso y, claro, el resultado al que se llega al utilizar Selenium. Creo que es un excelente añadido a nuestro catálogo de lecciones, sobre todo en la colección de Python.

Antes de pasar a la fase de revisión por pares (y la lectura por parte de un editor a la lección), me gustaría que hicieras algunos cambios que mejorarán la lección (dejo comentarios por secciones o párrafos):

Puedes realizar los cambios directamente en el archivo .md que ya tenemos aquí en el repositorio o en el tuyo en tu computadora y volvérnoslo a pasar.

Quedo pendiente para responder a cualquier duda que pueda surgir de este listado de sugerencias. ¡Mil gracias!

hernandezj1 commented 6 months ago

Muchas gracias por sus correciones. Estoy en el proceso de integrarlas. Una pregunta, ¿Ustedes tienen un formato específico para determinar una seccíon de Alerta en vez de un párrafo?

jenniferisasi commented 6 months ago

¡Hola @hernandezj1! Genial.

Sí, usamos este formato para alertas, que aparecerá en una "caja" de color amarillo (me parece) en el cuerpo de la lección:

<div class="alert alert-warning">
 Be sure that you follow directions carefully!
</div>

Cualquier otra duda, por aquí estoy para ayudarte. ¡Gracias!

hernandezj1 commented 6 months ago

Cree un fork para cambiar el texto poco a poco y lo pueden encontrar aquihttps://github.com/hernandezj1/ph-submissions/blob/gh-pages/es/borradores/originales/web-scraping-utilizando-selenium.md.

Unas preguntas:

  1. ¿Que parrafos son 26 y 29? , ya que no los pude localizar.
  2. Añadi la alerta pero no aparece formateada. Entiendo que eso aparecera despues en el website cuando se suba con los formatos de PH, ¿verdad?
anisa-hawes commented 6 months ago

Hola @hernandezj1,

Thank you for your note. We invite authors to make direct edits to their Markdown file within our repository /es/borradores/originales/web-scraping-utilizando-selenium.md. Would you feel comfortable working in this way?

Please let me know if you need any advice, or if you'd prefer us to pull your edits from your fork and apply them on your behalf.

You are correct that the <div class="alert alert-warning"> displays as raw html in GitHub, but it will be rendered as an alert box on our site (including in the Preview).

Thank you, Anisa

hernandezj1 commented 6 months ago

Oka, I will push the fork for the edits I have done and then apply the new ones on your copy. I must still ask about 26 and 29 because I cannot identify these paragraphs.

anisa-hawes commented 5 months ago

Hello @hernandezj1.

No need to push the edits from your fork – I've simply applied these as a direct commit: https://github.com/programminghistorian/ph-submissions/commit/69014f8a9dffe3f885750a6735a9fa67f8e098a0. I've also added your alert box above the Conclusion.

With these updates, some line and paragraph numbers have shifted but I believe @jenniferisasi's reference to Paragraph 26 is (what is now) line 146 of the Markdown file.

Reiteramos que nuestro ejemplo solamente utilizara Chrome, pero fácilmente puede hacer lo mismo con los demás browsers.

Jenn has suggested that this could be an alert box, rather than a paragraph. In this case, I think an information box (instead of a warning) might be a good option, and I've made this change: https://github.com/programminghistorian/ph-submissions/commit/809dd02deffbfb7d3e682763738faad7e67ab5f2

Paragraph 29 = Line 157 of the Markdown file

Aqui se debe notar también que driver. permite acceso a todos los métodos que tiene el driver de Selenium; desde interactuar con elementos específicos de la página hasta cerrar nuestro buscador. Por esta razon todas las instrucciones que siguen, excepto las de procesamiento de data, usan driver. antes del método.

Let us know where you think this could be best placed or mentioned earlier?

To avoid knots between forks, we encourage you to make further edits directly to the Markdown file within our repository /es/borradores/originales/web-scraping-utilizando-selenium.md. I'm happy to help at any time.

Thank you, Anisa

hernandezj1 commented 5 months ago

Hello,

Thanks for the clarifications and for adding the alert. I will be completing the last of the changes directly into the markdown before January 7th. Happy holidays and thank you for your help and patience in this process!

jenniferisasi commented 4 months ago

Thank you, @hernandezj1! I will take another look at all and assign this lesson an editor as soon as we can.

jenniferisasi commented 2 months ago

Estimado @hernandezj1, disculpa la tardanza. Yo misma actuaré de editora principal de la lección y estoy buscando revisores. Espero dar con un par de personas rápido para agilizar el proceso.

jenniferisasi commented 1 month ago

@hernandezj1, disculpa de nuevo, sigo en búsqueda de revisores.

jenniferisasi commented 1 week ago

Estimado @hernandezj1, ¡tengo buenas noticias! Nuestro buen amigo @jairomelo ha acordado actuar de primer revisor (experto) en tu trabajo. @jairomelo mi gracias por acceder.

Creo que ya conoces de sobra nuestras guías y expectativas para los revisores pero si tienes cualquier pregunta no dudes en ponerte en contacto con nosotros. Esperamos leer pronto (4-6 semanas) tus comentarios.

¡Gracias!

hernandezj1 commented 1 day ago

Muchas gracias @jenniferisasi por su constante comunicacion y trabajo para revisar esta leccion. Dejenme saber cualquier cosa que necesiten en el proceso @jenniferisasi y @jairomelo .