-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Balance] Changes to several trainer classes #5505
base: beta
Are you sure you want to change the base?
Conversation
Some changes I'd make
Lastly could you confirm that the methods to spawn mons based on movepool/abilities/etc. leave out cover legends and the like? I don't think anyone's hyped to fight a Parasol Lady with a Lugia... |
Weather Ball was supposed to catch Roserade but SwSh got rid of it... |
Cud Chew imo is in the same boat as Gluttony where sure it's not on food mons but the ability itself is heavily food-themed. If we're gonna put stuff like Grumpig and the elemental monkeys in through Gluttony we can live with a Fire Tauros and a Farigiraf. But even moreso than Cud Chew I want to reiterate Gulp Missile, this is a mon that tries to eat other stuff during battle. Also reminded me that Fillet Away is a thing, could be added too. Sure it's not "baking" but Teatime and Milk Drink and the like aren't directly tied to baking either, just food as a whole. |
Sorry sir this is a seafood-free bakery |
What are the changes the user will see?
**Biker**
*The Biker previously chose random Poison-types. They now have a proper pool of species they pull from.***Beauty**
*The Beauty previously had nothing defined regarding party members.*Dancer now has Quaxly and Jangmo-o instead of Popplio in Super Rare
**Cyclist**
**Baker**
Previously, the Baker would use Grass- or Fire-types.
Baker will now use mons which match any of these criteria:
**Parasol Lady**
Previously, Parasol Lady would just get random Water-types.
Parasol Lady now uses mons which match any of the following criteria:
The following Trainers have been moved around:
Why am I making these changes?
What are the changes from a developer perspective?
Screenshots/Videos
How to test the changes?
Checklist
beta
as my base branchnpm run test
)npm run create-test
) or updated existing tests related to the PR's changes?Are there any localization additions or changes? If so: