Skip to content

refactor pokemon and item data#213

Open
joos-too wants to merge 23 commits into
masterfrom
211-cleanup-pokemon-entries
Open

refactor pokemon and item data#213
joos-too wants to merge 23 commits into
masterfrom
211-cleanup-pokemon-entries

Conversation

@joos-too
Copy link
Copy Markdown
Owner

@joos-too joos-too commented Apr 27, 2026

  • see attached issues for change log
  • please review/test thoroughly because there are a lot of changes
  • please pay special attention to how missing images for custom pokemon names/items currently look
    -> needs a better solution i guess

After merging: import migrated database json to firebase!

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
@joos-too joos-too self-assigned this Apr 27, 2026
@joos-too joos-too linked an issue Apr 27, 2026 that may be closed by this pull request
8 tasks
Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
…licity

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
…ost places

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
@joos-too joos-too linked an issue Apr 28, 2026 that may be closed by this pull request
6 tasks
@joos-too joos-too changed the title refactor pokemon data and change to id based saving of pokemon refactor pokemon and item data Apr 28, 2026
… use pokedex-promise-v2

- working in progress on translating new evolutions generated from generate-pokemon.mjs
- renamed generate-pokemon-de.mjs to generate-pokemon.mjs and added all scripts to package.json

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
- add translations for items and some location fallback logic

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
@FreakMediaLP FreakMediaLP requested review from FreakMediaLP and removed request for FreakMediaLP April 28, 2026 16:38
@joos-too joos-too marked this pull request as draft April 29, 2026 13:28
joos-too added 10 commits April 30, 2026 11:31
… data via new pokeapi-static-client.mjs

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
… fix readme

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
- remove alt texts in many places where they are not needed because they lead to unwanted visual inconsistencies
- fix pokemon sprites in DeleteLinkModal.tsx and SelectLossModal.tsx to use generationSpritePath

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
@joos-too joos-too requested a review from FreakMediaLP May 17, 2026 14:48
@joos-too joos-too marked this pull request as ready for review May 17, 2026 14:48
@FreakMediaLP
Copy link
Copy Markdown
Collaborator

FreakMediaLP commented May 18, 2026

(With custom tracker enabled) If you add an Item, by simply entering its name, without selecting it, it wont be detected correctly.
image

For Pokémon it works as intended, as soon as you enter the correct name (even with custom tracker off), it automatically selects the Pokémon and shows the Sprite
image

@FreakMediaLP
Copy link
Copy Markdown
Collaborator

The custom-tracker-toggle is way to big in my oppinion, it should not be bigger than the version and ruleset selection, maybe just a small text (left) and a toggle (right), same as on the settings page.
Also I would rename it, so sth like "Version Override" (sounds cooler and is shorter), also the current naming could be missleading, so someone would think, that without this setting on, only the pokemon from exacly that generation would be available.
image

@FreakMediaLP
Copy link
Copy Markdown
Collaborator

FreakMediaLP commented May 18, 2026

Side note, all three item-categories use the same tooltip (evolution stone), rename it to "Item".
Also for consistency, maybe rename the "Stones" category (in this modal) to "Evolution Stones", as we currently have "Stones" and "Mega Stones"
image

@FreakMediaLP
Copy link
Copy Markdown
Collaborator

FreakMediaLP commented May 18, 2026

I dont know what is happening here, would like some info for better understanding, are this Gen 7+ Items, which dont have sprites in PokéAPI yet?
image

Copy link
Copy Markdown
Collaborator

@FreakMediaLP FreakMediaLP left a comment

Choose a reason for hiding this comment

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

Check comments (pokemon data cleanup looks good so far, could not find an issue)

Edit: Found some ↓↓↓

Comment thread src/data/pokemon.ts
Comment thread src/data/pokemon.ts
Comment thread src/services/evolutionMethodFilter.ts
Comment thread src/services/evolutionMethodFilter.ts
@FreakMediaLP
Copy link
Copy Markdown
Collaborator

FreakMediaLP commented May 18, 2026

In ORAS, Nosepass evolution is supposed to be in "New Mauville", but it shows "Thunder Stone", which was added in Gen8
image
image

Edit: Exact same issue for "Magneton -> Magnezone in ORAS", only that it dosnt show any evolution method at all
image

@FreakMediaLP
Copy link
Copy Markdown
Collaborator

Edit: Exact same issue for "Magneton -> Magnezone in ORAS", only that it dosnt show any evolution method at all <img

Maybe check all evolution methods again, I just checked Eevee and these two and all had something wrong

@joos-too
Copy link
Copy Markdown
Owner Author

joos-too commented May 18, 2026

In ORAS, Nosepass evolution is supposed to be in "New Mauville", but it shows "Thunder Stone", which was added in Gen8
image
image

Edit: Exact same issue for "Magneton -> Magnezone in ORAS", only that it dosnt show any evolution method at all
image

I'll check filters again. Probably something wrong.
Edit: Forget to filter it out because it wasn't in this List: https://www.pokewiki.de/Liste_der_speziellen_Entwicklungen.

@joos-too
Copy link
Copy Markdown
Owner Author

joos-too commented May 18, 2026

Edit: Exact same issue for "Magneton -> Magnezone in ORAS", only that it dosnt show any evolution method at all <img

Maybe check all evolution methods again, I just checked Eevee and these two and all had something wrong

The missing locations for ORAS is a pokeapi data problem in my opinion, so best to open a pr there, because pokemon.ts is generated and shouldn't be edited. They are probably all missing because Evolution Locations like that didn't exist back in gen 3 and they didn't add them afterwards.

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
@FreakMediaLP
Copy link
Copy Markdown
Collaborator

Edit: Forget to filter it out because it wasn't in this List: https://www.pokewiki.de/Liste_der_speziellen_Entwicklungen.

They are in the list, or did I miss something?
image

@joos-too
Copy link
Copy Markdown
Owner Author

Edit: Forget to filter it out because it wasn't in this List: https://www.pokewiki.de/Liste_der_speziellen_Entwicklungen.

They are in the list, or did I miss something?
image

Yes i meant the Gen 8 Stone Evolution i filtered out. nosepass is missing there. I only used this list to adjust filters and didn't check all evo locations.

@FreakMediaLP
Copy link
Copy Markdown
Collaborator

Change made, waiting for approval PokeAPI/pokeapi#1525

joos-too added 4 commits May 23, 2026 17:16
…ally resolve item slugs

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
…ion tooltips

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
… all pokemon in all games

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
…alization, toggle size adjustments and adding it to settingspage

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
@joos-too joos-too requested a review from FreakMediaLP May 23, 2026 16:31
@joos-too
Copy link
Copy Markdown
Owner Author

I made the suggested changes and also added generation sprite fallbacks for all gens.
This is needed if somebody is combining generation specific sprites with version override setting so sprites aren't missing.

We should also think about, if we want to merge this pr before the pokeapi changes are live (and deploy the pokemon.ts changes some other time) or if we wait.

After merging: import migrated database json to firebase!

Signed-off-by: Jan L <80320565+joos-too@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

itemTracker followup cleanup pokemon data

2 participants