Skip to content
This repository was archived by the owner on Jul 11, 2026. It is now read-only.

Pokémon Guide#42

Open
TangCherry wants to merge 40 commits into
Laboratoria:masterfrom
TangCherry:master
Open

Pokémon Guide#42
TangCherry wants to merge 40 commits into
Laboratoria:masterfrom
TangCherry:master

Conversation

@TangCherry

Copy link
Copy Markdown

No description provided.

@HectorBlisS

Copy link
Copy Markdown
Contributor

Tengo la sensación de que faltan muchos cambios? y me hace mucho ruido que solo 1 de ustedes tenga nombre y foto (cuenta) revisen eso porque si no todo el trabajo es anónimo. ;)

Comment thread docs/data.js
import data from './data/pokemon/pokemon.js';

//funcion para mostrar todos los pokemones
export function showPokemons (pokemon) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Recuerda ser más semantica, la función recibe 1 pokemon y devuelve un array de pokemones? tal vez recibe un ID?, sé más descriptiva ;)

Comment thread package.json
"@babel/core": "^7.6.2",
"@babel/plugin-transform-modules-commonjs": "^7.6.0",
"babel-jest": "^24.9.0",
"express": "^4.17.1",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

porqué está express instalado? XD

Comment thread src/data.js
Comment on lines +3 to +17
//funcion para mostrar todos los pokemones
export function showPokemons (pokemon) {
let allPokemons = data.pokemon
for (let i = 0; i <= allPokemons.length; i++){
}
return allPokemons;
}



//funcion para filtrar por tipo
export function filterByType(type){
let filter = data.pokemon.filter(poke=>poke.type.includes(type))
return filter;
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Este código ya vivía en otro archivo

Comment thread docs/main.js
let weight = document.createElement ("p");
let candyCount = document.createElement ("p");
let egg = document.createElement ("p");
let weakness = document.createElement("p");

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Muy bien el orden de los nodos

Comment thread src/data.js
@@ -14,4 +14,5 @@ export function showPokemons (pokemon) {
export function filterByType(type){

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

código que ya existe

Comment thread src/style.css Outdated
}

.item-flex:hover {
color:red;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sean más consistentes con los espacios

Comment thread src/data.js
// import data from './data/lol/lol.js';
// import data from './data/patient/patient.js';
// import data from './data/pokemon/pokemon.js';
import data from './data/pokemon/pokemon.js';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Los comentarios de los commits no son descriptivos, mejorarlos.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants