Skip to content

Conversation

@jemarq04
Copy link
Contributor

@jemarq04 jemarq04 commented Apr 8, 2025

Add entries in location_names.csv for missing location IDs. Only English names are added in these commits. Additionally, a couple typos were fixed in locations.csv: 'cemetary' to 'cemetery', 'mauville-space-center' to 'mossdeep-space-center', and 'spike-decision' to 'split-decision'. Lastly, 'naranja-academy/uva-academy' was split to 'naranja-academy' and 'uva-academy', since these are distinct locations.

The English names were retrieved using a script that queries Bulbapedia with the name in locations.csv - any pages that were not redirected to their appropriate page were corrected manually. For reproducibility, the URL used was "https://bulbapedia.bulbagarden.net/wiki/Special:Search/NAME" with "NAME" replaced to each name in locations.csv that was not present in location_names.csv (e.g. 'split-decision-ruins').

Edit: I also added missing locations within Kitakami and Blueberry Academy from the Scarlet/Violet DLC. Once again, this used Bulbapedia:

@jemarq04 jemarq04 changed the title Add English names for missing location name entries Add English names for missing location name entries and missing Kitakami and Blueberry locations Apr 8, 2025
@Naramsim
Copy link
Member

Naramsim commented Apr 9, 2025

Hi, the build system had some problems, could you add a commit to the PR or close/reopen it?

@jemarq04
Copy link
Contributor Author

jemarq04 commented Apr 9, 2025

Hi, the build system had some problems, could you add a commit to the PR or close/reopen it?

Ok, that should've worked. I'm not sure why there were problems as this branch modified different files, but after pulling the master branch back into it everything seems ok now. I'll monitor these checks.

Copy link
Member

@phalt phalt left a comment

Choose a reason for hiding this comment

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

Great contributions, thank you.

1054,5,roaming-unova
1055,3,mossdeep-space-center
1056,6,roaming-kalos
1057,9,uva-academy
Copy link
Member

Choose a reason for hiding this comment

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

Nice

901,9,Giant's Foot,
902,9,Giant's Mirror,
903,9,Giant's Seat,
904,9,503 Backend fetch failed,
Copy link
Member

Choose a reason for hiding this comment

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

👀 surprised this got in ha

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My heart dropped when I almost left that in the PR! Running the script again resolved it, so I'm unsure how it happened. I'll definitely be on the lookout for things like that in the future haha

@Naramsim Naramsim merged commit 9bd1a6e into PokeAPI:master Apr 10, 2025
8 checks passed
@pokeapi-machine-user
Copy link

A PokeAPI/api-data refresh has started. In ~45 minutes the staging branch of PokeAPI/api-data will be pushed with the new generated data.

The staging branch will be deployed in our staging environment and the entire API will be ready to review.

A Pull Request (master<-staging) will be also created at PokeAPI/api-data and assigned to the PokeAPI Core team to be reviewed. If approved and merged new data will soon be available worldwide at pokeapi.co.

@pokeapi-machine-user
Copy link

The updater script has finished its job and has now opened a Pull Request towards PokeAPI/api-data with the updated data.

The Pull Request can be seen deployed in our staging environment when CircleCI deploy will be finished (check the start time of the last build).

Naramsim pushed a commit to PokeAPI/api-data that referenced this pull request Apr 10, 2025
@jemarq04 jemarq04 deleted the add_location_names branch April 10, 2025 11:49
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.

4 participants