-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add more pokemon! #1
Comments
I'm gonna look at it! |
it's fun adding pokemon ^^ |
#6 :) |
Why add Pokemon manually, when you can actually use the provided API to get all the data you want using 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 |
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/ |
#10 Added one more |
@Juggernaut9 I added 40 Pokemon please merge it :) |
I m gonna try this. Seems fun |
@Juggernaut9 Can I work on and contribute to this? |
@Juggernaut9 already added some if you want to merge. #46 |
@Juggernaut9 can I contribute to this? |
yes you can @krishnasreekoganti |
I raised a PR for new Pokemon @Juggernaut9 can you please review |
@Juggernaut9 please accept my PRs for hacktoberfest |
Please accept my PRs for Hacktoberfest...!! |
Hey @BhavyaCodes!! I added 5 Pokémon for Hacktoberfest, but Vercel build failed. PR: #74 |
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
The text was updated successfully, but these errors were encountered: