Skip to content

Conversation

joelte
Copy link
Member

@joelte joelte commented Sep 9, 2026

Added a grid of pokemon with images and names that update with the filter.
Filter is now saved with html storage.
Gens and Types now pulled directly from PokeAPI to ensure every aspect of the grid and filter remains up to date.

closes #3
closes #5

joelte added 2 commits September 9, 2026 22:15
Added a grid of pokemon pulling the name and image from the PokeAPI api.
Added filter fundtionality for the grid of pokemon cards. Filter now saves with html storage and pulls names from PokeAPI to autoupdate incase of new added gens or types.
@joelte joelte added this to the Delivery 1 milestone Sep 9, 2026
@joelte joelte marked this pull request as ready for review September 9, 2026 20:54
@joelte joelte requested a review from a team as a code owner September 9, 2026 20:54
Copy link
Member

@halvortf halvortf left a comment

Choose a reason for hiding this comment

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

Godt du har med error checking!
Har du mulighet til å kommentere mer i koden? Slik av vi dødelige kan forstå mer av det som er gjort! :)
Noe vi må ta med oss videre:

  • Akkurat nå plukker vi pokemon som var nye til hver generasjon
  • Eldre pokemon kan faktisk ha vært med i den nye generasjonen, selv om de ikke er strengt talt nye
  • Eksempelvis, Zubat og Golbat (som er samme pokemon) som ble introdusert i 1. generasjon, og Crobat (fortsatt samme pokemon) ble introdusert i 2. generasjon.
  • Hvis vi velger "Gen II" og "Poison", så kommer bare Crobat opp. Er det noe mulighet med API'en å få opp de tidligere evolusjonene av Crobat?
  • Evt. annen løsningen kan være å plukke ut de siste evolusjonene til hver pokemon når vi faktisk skal generere laget.
  • Én pokemon kan ha to typings (eksempelvis, Fire + FIghting). Hadde det vært kult å vise typings til de som dukker opp også?

joelte added 3 commits September 12, 2026 20:00
Added another filter to see pokemon from a specific game.
Added some comments to explain some of the code snippets that are not self explanitory
Added icons for the types for every pokemon by pulling their id and fetchign an icon from a github repo for pokemon
@joelte
Copy link
Member Author

joelte commented Sep 12, 2026

Godt du har med error checking! Har du mulighet til å kommentere mer i koden? Slik av vi dødelige kan forstå mer av det som er gjort! :) Noe vi må ta med oss videre:

* Akkurat nå plukker vi pokemon som var nye til hver generasjon

* Eldre pokemon kan faktisk ha vært med i den nye generasjonen, selv om de ikke er strengt talt nye

* Eksempelvis, Zubat og Golbat (som er samme pokemon) som ble introdusert i 1. generasjon, og Crobat (fortsatt samme pokemon) ble introdusert i 2. generasjon.

* Hvis vi velger "Gen II" og "Poison", så kommer bare Crobat opp. Er det noe mulighet med API'en å få opp de tidligere evolusjonene av Crobat?

* Evt. annen løsningen kan være å plukke ut de siste evolusjonene til hver pokemon når vi faktisk skal generere laget.

* Én pokemon kan ha to typings (eksempelvis, Fire + FIghting). Hadde det vært kult å vise typings til de som dukker opp også?

Har lagt inn noen kommentarer for ikke selvforklarende kode.
La till ikoner for å vise typings.
Det er ingen info i APIen om cross generation evolutions, men vi får uansett litt samme funksjonalitet av å sortere etter spill.

@joelte joelte requested a review from halvortf September 12, 2026 22:38
Copy link
Member

Choose a reason for hiding this comment

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

Nice!! Funker akkurat som jeg så for meg. Veldig bra

Copy link
Member

@halvortf halvortf left a comment

Choose a reason for hiding this comment

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

Vi kan fjerne Generation , tror det blir ryddigst hvis bruker velger bare ut ifra spill og type. Bildene ser utrolig bra ut! Artig med typing icon på hver Pokemon!

Removed the filter for generation, now pokemon can only be sorted by type and game.
@joelte joelte requested a review from halvortf September 14, 2026 10:55
@joelte
Copy link
Member Author

joelte commented Sep 14, 2026

Removed the filter for generations

Copy link
Member

@halvortf halvortf left a comment

Choose a reason for hiding this comment

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

Nice! Dette er bra.

@halvortf halvortf merged commit 2299144 into dev Sep 15, 2026
@joelte joelte deleted the Pokemoncards branch September 16, 2026 08:36
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants