-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Bug] Elite Four for Gen 8 is missing, so Leon is messing up the Elite Four -> Champion order for Gen 9 #1118
Comments
Well according to https://bulbapedia.bulbagarden.net/wiki/Galar_League you fight always Marnie, Hop, (Bede - he is officially not part so i dont know if we include it), Nessa, (Bea or Allister depending on the game), Raihan. So it would be 5 fights even if we cut Bede (which we should...). I would also Cut Hop so the elite 4 all have a typ theme. So it would be Marine (Dark), Nessa (Water), Bea(Fighting)/Allistar(Ghost), Raihan (Dragon). Whats your opinion about that? |
That sounds fine, I don't know about the type theme tho, I would avoid reusing gym leaders in elite four whenever possible to add more variety and potentially more egg vouchers |
Have a look at Larry and or the dragon guy from kanto (dont really know the english name...). They have seperate trainer classes for their second apparence. Same could be done for the gym leaders of galar |
* main: (43 commits) Added localization to the biomes. (pagefaultgames#1378) Fix French translation of Genesect's Drive (pagefaultgames#1379) Adding clause so right Pokemon in double battle cannot target itself with a spread move when left one dies with no replacement (pagefaultgames#1370) Fix incorrectly unselectable starter formes (pagefaultgames#1332) Made Lab 25% instead of 12.5% to encounter after Factory (pagefaultgames#1351) Fixed German localization for Starf Berry description and localized zippyZap effect (pagefaultgames#1373) Swapped Aron for Aggron on Iris pokemon pool (pagefaultgames#1374) Adds New BattleSceneEventTypes (pagefaultgames#1372) localized Honeygather again (pagefaultgames#1369) hotfix for alder bgm on double battle (pagefaultgames#1368) Implement Honey Gather (pagefaultgames#1360) Adding the option to have named trainers be able to have a double battle together (pagefaultgames#1318) Make iv scanner check against baby form IVs (pagefaultgames#1214) Issue pagefaultgames#1118 Added Galar Elite 4 (pagefaultgames#1246) Beat Up checks the user's party instead of always checking player's party (pagefaultgames#1268) Implement Quick Guard and other conditional team protection moves (pagefaultgames#1275) Implement Purify (pagefaultgames#1291) Update move.ts (pagefaultgames#1359) Fix Wind Rider (pagefaultgames#607) Implement Poison Puppeteer (pagefaultgames#320) ...
This is likely caused because in the code in src/battle.ts, line 326, the array for champions lists 10 options, while the elite four arrays only list 9 options.
In other words, Leon is just messing up the whole thing.
I didn't make a Pull Request on this issue because there's many ways to fix this, and I'm not sure how you want to implement it. But I'd suggest just adding an elite four for Gen 8, with Marnie, Hop, Bede and a random choice between a few Gym leaders, like it happens in the real game. Or just add that bad steel type guy. But I don't think Leon should be polluting the pools of other generations.
EDIT: Also, Kieran has no dialogue, unlike the other champions.
The text was updated successfully, but these errors were encountered: