Skip to content
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

Changed Scale Shot Effect Text #758

Merged
merged 4 commits into from
Oct 1, 2022
Merged

Conversation

norman-codes
Copy link
Contributor

CONTENT

The effect text for Scale Shot (move ID 799) was modified to be more accurate and specific, in accordance with Issue #749.

As part of doing so, 3 files were modified:

  1. move_effects.csv
    The new effect_id 443 was added.

  2. moves.csv
    The effect_id of scale_shot was changed from 1 to 443.

  3. move_effect_prose.csv
    The new effect text under the ID 443 was added.

TESTING

For testing, the instructions under "Setup" and "Database setup" were followed and the API was locally hosted.

Then, a GET request was made by accessing the Scale Shot move page in my browser:
(localhost IP)/api/v2/move/scale-shot/

norman-codes and others added 2 commits September 30, 2022 15:22
The effect text for Scale Shot (move ID 799) was modified to be more accurate and specific, in accordance with Issue PokeAPI#749.

As part of doing so, 3 files were modified:
1. move_effects.csv
The new effect_id 443 was added.

2. moves.csv
The effect_id of scale_shot was changed from 1 to 443.

3. move_effect_prose.csv
The new effect text under the ID 443 was added.
7,9,Has a $effect_chance% chance to [paralyze]{mechanic:paralysis} the target.,Inflicts [regular damage]{mechanic:regular-damage}. Has a $effect_chance% chance to [paralyze]{mechanic:paralyze} the target.
8,9,User faints.,"User [faint]{mechanic:faint}s, even if the attack [fail]{mechanic:fail}s or [miss]{mechanic:miss}es. Inflicts [regular damage]{mechanic:regular-damage}."
9,9,Only works on sleeping Pokémon. Drains half the damage inflicted to heal the user.,[Fails]{mechanic:fail} if not used on a [sleep]{mechanic:sleep}ing Pokémon. Inflicts [regular damage]{mechanic:regular-damage}. [Drains]{mechanic:drain} half the damage inflicted to heal the user.
move_effect_id,local_language_id,short_effect,effect,
Copy link
Member

Choose a reason for hiding this comment

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

Hi, why did you add all the ,?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it's because I edited the .csv file in Microsoft Excel and it automatically did that. I'm sorry - I should have opened it in a text editor instead and only added that so that no formatting issues like that occurred. I'll update this PR and remove the unnecessary changes.

Fixed the comma formatting error caused by editing the .csv file using Microsoft Excel instead of a text editor.
@Naramsim Naramsim merged commit 87f2ecf into PokeAPI:master Oct 1, 2022
@Naramsim
Copy link
Member

Naramsim commented Oct 1, 2022

Thanks!

@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 you will be able to review the entire API.

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.

You can see the Pull Request deployed at our staging environment when CircleCI deploy will be finished (check the started time of the last build).

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.

3 participants