BhavyaCodes / pokemon

https://pokemon-hacktoberfest.vercel.app
15 stars 40 forks source link

Add more pokemon! #1

Open BhavyaCodes opened 3 years ago

BhavyaCodes commented 3 years ago

Go to pokeapi website, Right below the Try It now heading you will find an api endpoint that you can try. replace ditto with your favorite pokemon, then edit /src/pokemon.json file. Add an object with the pokemon details in the array, be sure to get the id from pokeapi, you may use any imageurl, formatting should be exactly like the 2 original examples on top.

EDIT: please do not add more than 5 pokemon within a pull request! Current pull requests with upto 50 pokemon will be merged but more than that I'm sorry, I can't merge that, we don't have any system for pagination yet, also the data is saved in just a json file. It won't be efficient to import 1000 pokemon from json then sort them and render them on the DOM. This will also affect the search bar performance a lot

jakubkoje commented 3 years ago

I'm gonna look at it!

zatch3301 commented 3 years ago

it's fun adding pokemon ^^

redrails commented 3 years ago

https://github.com/Juggernaut9/pokemon/pull/6 :)

ghost commented 3 years ago

Why add Pokemon manually, when you can actually use the provided API to get all the data you want using fetch?

fetch(`https://pokeapi.co/api/v2/pokemon/${pokemonName}`)
  .then(response => response.json())
  .then(data => {
    // use pokemon data to display it
  });
RidhikGovind commented 3 years ago

Why add Pokemon manually, when you can actually use the provided API to get all the data you want using fetch?

fetch(`https://pokeapi.co/api/v2/pokemon/${pokemonName}`)
  .then(response => response.json())
  .then(data => {
    // use pokemon data to display it
  });

This is a great way to do things

BhavyaCodes commented 3 years ago

hey, thanks for the suggestions but this project is mainly focused at beginners so they may get some practice with contributing to open source. I have already made a similar app that uses fetch, you can check it here https://pokemon-alpha.vercel.app/

vatsalsharma376 commented 3 years ago

10 Added one more

zatch3301 commented 3 years ago

@Juggernaut9 I added 40 Pokemon please merge it :)

imabhinavagarwal commented 3 years ago

I m gonna try this. Seems fun

WulfPlasma commented 3 years ago

@Juggernaut9 Can I work on and contribute to this?

WulfPlasma commented 3 years ago

@Juggernaut9 already added some if you want to merge. #46

krishnasreekoganti commented 3 years ago

@Juggernaut9 can I contribute to this?

BhavyaCodes commented 3 years ago

yes you can @krishnasreekoganti

krishnasreekoganti commented 3 years ago

I raised a PR for new Pokemon @Juggernaut9 can you please review

daksha-aeer commented 3 years ago

@Juggernaut9 please accept my PRs for hacktoberfest https://github.com/Juggernaut9/pokemon/pull/67 https://github.com/Juggernaut9/pokemon/pull/68 https://github.com/Juggernaut9/pokemon/pull/69 https://github.com/Juggernaut9/pokemon/pull/70

MeetPatel41011 commented 3 years ago

Please accept my PRs for Hacktoberfest...!!

Maria-Papa commented 1 year ago

Hey @BhavyaCodes!!

I added 5 Pokémon for Hacktoberfest, but Vercel build failed. I can't see what's wrong with it, please tell me if anything needs fixing 😄!

PR: #74