CodeReview #26

EtnaSI · 2020-03-14T01:03:46Z

Falta depurar código y hacer pruebas.

HectorBlisS

Muy bien las descripciones en los commits, lastima que están en español, evítenlo en el siguiente proyecto. Se nota el trabajo progresivo y duro y también el trabajo en equipo, felicidades por el producto final, muy completo.

HectorBlisS · 2020-03-25T06:35:44Z

test/data.spec.js

@@ -128,7 +128,7 @@ escribe('data.orderNumber', () => {
  });

  it('should return Bulbasur first and Ivysaur after', () => {
-    expect(pokemon).toHaveProperty('weaknesses', 'Ice');
+    expect(pokemon.id).toBe('1' , '2');


muy bien por los tests

HectorBlisS · 2020-03-25T06:36:33Z

src/data.js

@@ -1,16 +1,16 @@
 import pokemon from './data/pokemon/pokemon.js';
- //console.log(pokemon);
+// console.log(pokemon);

 const data = {
  saveNumber: (searchNumber) => {


Muy bien los nombres de las funciones en ingles

HectorBlisS · 2020-03-25T06:36:58Z

src/data.js

@@ -1,16 +1,16 @@
 import pokemon from './data/pokemon/pokemon.js';
- //console.log(pokemon);
+// console.log(pokemon);

 const data = {


No es necesario crear un objeto, se pueden exportar las funciones e forma independiente

HectorBlisS · 2020-03-25T06:37:25Z

src/data.js

        orderFilter = pokemon.sort((a, b) => {
          if (a.name > b.name) {
            return 1;
          }


Se entiende porque se hace esto evitando el ordenado default unicode?

HectorBlisS · 2020-03-25T06:37:39Z

test/data.spec.js

@@ -1,5 +1,5 @@
 // importamos la función `example`
-import { data } from "../src/data";
+import { data } from '../src/data';


HectorBlisS · 2020-03-25T06:38:46Z

src/style.css

@@ -0,0 +1,234 @@
+@import url('https://fonts.googleapis.com/css?family=Saira+Condensed&display=swap');


Bien por el uso de fuentes externas

HectorBlisS · 2020-03-25T06:40:11Z

src/main.js

+      let overlay = document.querySelector('#profileContainer');
+      let popup =  document.getElementsByClassName('profileCard');


Preferible usar querySelector, ya que getElementsByClassName devuelve un tagList (lista extraña que no es un array)

HectorBlisS · 2020-03-25T06:40:39Z

src/main.js

      let newPopup = popup[popup.length-1];
      //console.log(newPopup);
-      let btnClosePopup = document.getElementsByClassName('btn-cerrar-popup')
+      let btnClosePopup = document.getElementsByClassName('btn-cerrar-popup');
      let newBtnClose = btnClosePopup[btnClosePopup.length-1];


Si buscamos un lemeno en especifico es mejor darle un id

HectorBlisS · 2020-03-25T06:40:52Z

src/main.js

          overlay.classList.remove('active')
          newPopup.classList.remove('active')


Bien el uso de classList

HectorBlisS · 2020-03-25T06:41:05Z

src/main.js

+const profileContainer = document.createElement("div");
+profileContainer.className = "profileContainer";
+profileContainer.id = "profileContainer";
+document.body.appendChild(profileContainer);


EtnaSI and others added 30 commits March 11, 2020 08:47

Se agregó HTML

498bfd7

Se agregó CSS

962161d

Se agregó BD y JS

1c91a70

Index

291d95b

Merge branch 'master' of https://github.com/EtnaSI/CDMX009-Data-Lovers

c7fe2e1

Se integró main y popup

0617c06

Se modificó boton select Ordenar

6a2593c

Se modificó estilo del select Ordenar

7fb26ba

A

2f8e7e3

Ya todo funcionando, falta depurar código

f9a3262

Hacer funcionar popups

fe63a16

Funcionando, falta depurar código

48f0a08

Update README.md

a174687

Update README.md

b30c08c

Se agregaron imágenes de README

e7bb4c5

Listo README

c5949c6

Update README.md

47ab0b2

Imagenes

78ac729

Update README.md

f205589

Update README.md

07c0547

Imágenes README

86f4f1f

Update README.md

9c789a6

Update README.md

e642d17

Delete historiasDeUsuario1.png

2c9daa1

Delete historiasDeUsuario2.png

42e52c8

Delete historiasDeUsuario3.png

f9d00f7

Delete historiasDeUsuario4.png

07bad53

Delete historiasDeUsuario5.png

8a6b091

Update README.md

7ef740c

Update README.md

6e05300

EtnaSI and others added 27 commits March 17, 2020 13:28

Delete historias3-4.png

87baa1f

Delete historias5-6.png

fe1a5ec

Add files via upload

fc6fa1e

Update README.md

01f9050

Update README.md

a5b72ff

Update README.md

c9fa370

Update README.md

55a1575

Cambios al README

dd80434

Por ver el pull

e672262

Se redujo main, se agregró docs

ac91867

Código depurado, pruebas

922fa86

Merge branch 'master' of https://github.com/EtnaSI/CDMX009-Data-Lovers

11e69a4

Faltaba un div para acomodar las cards

7a4c438

Merge branch 'master' of https://github.com/EtnaSI/CDMX009-Data-Lovers

7584b91

Se agrega div faltante

9ca3575

Se ajustó el diseño de las tarjetas

80d5077

Update README.md

49b1d81

Update README.md

6006e8c

Update README.md

b90af45

Se actualizó select Debilidades

69853f2

Se agregan opciones faltantes al select Debilidades

f86a598

Se agregó footer

421f762

Corrección de sintaxis

dd42167

Se corrigió link imagen pokemon

32386a5

Se corrigió select weaknesses en docs

aa16a69

Se depuró sintaxis código

33239d3

Se corrigió select type en docs

cbdd1c6

HectorBlisS suggested changes Mar 25, 2020

View reviewed changes

EtnaSI added 2 commits August 22, 2020 11:31

Imagen WebSite

084860d

Add files via upload

ef45d43

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

CodeReview #26

CodeReview #26

EtnaSI commented Mar 14, 2020

HectorBlisS left a comment

HectorBlisS Mar 25, 2020

HectorBlisS Mar 25, 2020

HectorBlisS Mar 25, 2020

HectorBlisS Mar 25, 2020

HectorBlisS Mar 25, 2020

HectorBlisS Mar 25, 2020

HectorBlisS Mar 25, 2020

HectorBlisS Mar 25, 2020

HectorBlisS Mar 25, 2020

HectorBlisS Mar 25, 2020

		@@ -0,0 +1,234 @@
		@import url('https://fonts.googleapis.com/css?family=Saira+Condensed&display=swap');

		let overlay = document.querySelector('#profileContainer');
		let popup = document.getElementsByClassName('profileCard');

		overlay.classList.remove('active')
		newPopup.classList.remove('active')

CodeReview #26

Are you sure you want to change the base?

CodeReview #26

Conversation

EtnaSI commented Mar 14, 2020

HectorBlisS left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment