Skip to content

Update Item and EvolutionDetail to add new API changes - #49

Merged
mlemesle merged 6 commits into
mlemesle:mainfrom
jemarq04:feat/add-item-updates
Sep 20, 2026
Merged

mlemesle merged 6 commits into
mlemesle:mainfrom
jemarq04:feat/add-item-updates

Conversation

@jemarq04

@jemarq04 jemarq04 commented Aug 10, 2026 •

Copy link
Copy Markdown
Contributor

Updates Item struct and adds Currency and ItemPrice structs to match changes from PR PokeAPI/pokeapi#1431.

To the Item struct, removes the cost field and replaces it with a prices field. Because of the changes to the API, all queries for items have been failing as the cost field is no longer available.

Edit: Some more breaking changes were introduced with the API regarding evolution information (see PokeAPI/pokeapi#1656), so I added the changes here as well.

@mlemesle

Copy link
Copy Markdown
Owner

Hey @jemarq04 !

Thanks for another contribution !
I'll look into it when I'll come back from vacations

@jemarq04

Copy link
Copy Markdown
Contributor Author

Sounds good!

@jemarq04 jemarq04 changed the title Update Item to add new pricing/currency changes Update Item and EvolutionDetail to add new API changes Sep 20, 2026
@jemarq04

Copy link
Copy Markdown
Contributor Author

Updated the PR to match some more breaking changes from the API on evolutions. All queries on evolutions/items work when testing with this branch.

@mlemesle
mlemesle merged commit b4c8993 into mlemesle:main Sep 20, 2026
@mlemesle

Copy link
Copy Markdown
Owner

Hey @jemarq04,

Thanks for another great addition to the crate !

Apologies for the delay

@jemarq04
jemarq04 deleted the feat/add-item-updates branch September 20, 2026 19:22
@jemarq04

Copy link
Copy Markdown
Contributor Author

No worries, thanks! I’m on the PokeAPI team now so it’s easy to translate any schema changes to the wrapper. Happy to help!

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.

2 participants