Code Review Pendientes

[UlisesGascon]

¿De qué va esto?

Hola! La idea de este issue es agrupar todo lo que esta pendiente de ser revisado. En estos últimos días he recibido muchas notificaciones por Github, pero no siempre he sido capaz de volver a ellas una vez abierto el aviso en el email. 😓 Así que para asegurarme que no se me olvida ninguna os pediría que añadáis una respuesta a este issue con las revisiones que estáis esperando y las nuevas que vengan. 🙏

¿Cómo funciona esto?

Solo tienes que añadir tu respuesta a este issue y yo añadiré las revisiones en la Lista actual. También te avisaré cuando haya revisado todo 👍

Lista actual 🤓

Pendientes

• PR de ejercicio express-exercise-proxy de @minicatsCB
• 🔴 PR Ejercicioomdb-server-render de @minicatsCB
• PR Ejercicio de caesar-cipher de @minicatsCB
• PR Ejercicio Firebase de @gargonmi
• PR Proyecto document-manager-platform de @gargonmi
• 🔴 PR Ejercicio omdb-exercise de @minicatsCB
• 🔴 PR Ejercicio prioritario firebase-auth de @minicatsCB
• PR Ejercicio BuenosDiasMadrid de @minicatsCB.
• PR Ejercicio LocalStorageExample de @minicatsCB.
• PR Ejercicio AgendaContactos de @calope03
• PR Ejercicio Gráfica Weather Map de @codemcu
• PR Ejercicio Curiosity de @codemcu
• PR Ejercicio Curiosity de @minicatsCB
• PR Ejercicio Temperatura-Madrid de @calope03
• PR Proyecto VAMKA de @minicatsCB
• PR Ejercicio NewYorkTimesBooks2 de @minicatsCB
• PR Ejercicio MadridPollenData de @minicatsCB
• PR Ejercicio NewYorkTimesBooks1 de @minicatsCB
• PR Ejercicio VendingMachineProject de @minicatsCB
• PR Ejercicio moviefire de @codemcu

Por clarificar

• PR ejercicio NASA rover challenge de @gargonmi
• PR Ejercicio MovieFire de @alberto-gomez-munity

Resueltas

• PR ejercicio nodejs-router de @minicatsCB
• PR Ejercicio nodejs-earthquakes de @minicatsCB
• PR Ejercicio RouterAndTemplatesExercise de @minicatsCB
• PR Ejercicio expresiones regulares (Regex) de @minicatsCB
• PR Ejercicio ConsoleCommands1 de @minicatsCB
• PR Ejercicio Metro de @codemcu
• PR ejercicio terremotos de @codemcu
• PR Ejecricio agenda contactos de @codemcu
• 🔥🔥🔥 PR Proyecto flash-typer de @minicatsCB.
• PR Ejercicio GeolocationExercise de @minicatsCB
• PR Ejercicio OpenWeatherMap de @minicatsCB
• PR Ejercicio HttpLibrary de @minicatsCB
• PR Ejercicio NYTimes de @codemcu
• PR Ejercicio whereIAm de @gargonmi
• PR Proyecto precio-bitcoin de @Parzival33
• PR Proyecto primer-proyecto-master de @calope03
• PR ejercicio NASA rover challenge de @minicatsCB
• PR Proyecto Songkick Event Finder de @codemcu
• PR Ejercicio GithubSearchProject de @minicatsCB
• PR Proyecto piano project de @gargonmi
• PR Github Search de @codemcu
• PR ejercicio Curiosity de @gargonmi
• PR Ejercicio GithubSearchProject de @codemcu
• PR Ejercicio transporte_los_angeles de @calope03

[alberto-gomez-munity]

De momento no es más que la definición del proyecto, pero irá tomando forma:
#54

[codemcu]

te copio el pull request de mi proyecto 1, #33

[UlisesGascon]

✅ @calope03 ya tienes mi review del proyecto

[UlisesGascon]

✅ @minicatsCB ya tienes mi review de GithubSearchProject

[UlisesGascon]

✅ @minicatsCB ya tienes mi review de NASA rover challenge

[UlisesGascon]

✅ @gargonmi ya tienes mi review de Curiosity

[UlisesGascon]

✅ @codemcu ya tienes mi review de NYTimes

[UlisesGascon]

@minicatsCB y @gargonmi mñn en clase vemos como arreglar lso PRs porque no me esta dejando dar feedback como en los otros...😓

[UlisesGascon]

✅ @codemcu ya tienes mi review de GithubSearchProject

[calope03]

Pull request del ejercicio de los vehiculos de Los Angeles: calope03/transporte_los_angeles#1

[UlisesGascon]

✅ @calope03 ya tienes mi review de transporte_los_angeles

[minicatsCB]

Por mi parte ya tienes las nuevas pull requests. En teoría, ahora deberías poder ver el código entero.

GeolocationExercise
VAMKA
NewYorkTimesBooks2
MadridPollenData
NewYorkTimesBooks1
VendingMachineProject
OpenWeatherMap
RouterAndTemplatesExercise

[UlisesGascon]

Genial @minicatsCB!! Ya las añado en la lista y te voy avisando cuando lo tenga todo revisado ^^

[UlisesGascon]

✅ @gargonmi ya tienes mi review de whereIAm

[UlisesGascon]

✅ @minicatsCB ya tienes mi review de HttpLibrary

[UlisesGascon]

✅ @minicatsCB ya tienes mi review de GeolocationExercise

[UlisesGascon]

✅ @minicatsCB ya tienes mi review de OpenWeatherMap

[calope03]

Pull request de la temperatura y humedad de Madrid calope03/Temperatura-Madrid#1

[minicatsCB]

Añado pull request del ejercicio del Curiosity.

[minicatsCB]

Añado pull request del ejercicio de expresiones regulares (Regex). En realidad, he probado las expresiones en Regex101 y funcionan, pero a lo mejor se pueden optimizar de alguna forma.

[codemcu]

Agrego PR del ejercicio del Curiosity #70

[codemcu]

Agrego PR del ejercicio del Metro de L.A #71

[codemcu]

Agrego PR del ejercicio del gráfica del Weather Map #72

[calope03]

Agrego PR de la agenda de contactos calope03/AgendaContactos#2

[minicatsCB]

Añado PR de LocalStorageExample.

[minicatsCB]

Añado PR de BuenosDiasMadrid.

[minicatsCB]

Añado PR de ConsoleCommands1.

[minicatsCB]

Añado PR 🔴 prioritario firebase-auth.

[minicatsCB]

Añado PR 🔴 prioritario omdb-exercise.

[minicatsCB]

Añado PR 🔥 🔥 🔥 prioritario flash-typer.

[minicatsCB]

Añado PR de nodejs-earthquakes.

[minicatsCB]

Añado PR de nodejs-router.

[minicatsCB]

Añado PR de caesar-cipher.

[minicatsCB]

Añado PR de express-exercise (el del CORS).

[minicatsCB]

Añado PR 🔴 de omdb-server-render.

Lo marco como prioritario por aquello de que los demás ejercicios de MovieFire dependen de este. Para corregir lo máximo posible y arrastrar el menor número de fallos posibles.