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

Version 1.3.0 #2447

Merged
merged 186 commits into from
Nov 18, 2022
Merged

Conversation

AsparagusEduardo
Copy link
Collaborator

@AsparagusEduardo AsparagusEduardo commented Nov 10, 2022

Changelog preview

Added

Changed

Fixed

Pret merges

Cleanup

New Contributors

Full Changelog: expansion/1.2.0...expansion/1.3.0

Discord contact info

AsparagusEduardo#6051

BuffelSaft and others added 30 commits September 23, 2021 14:31
Port multihit system from the CFRU, implement Parental Bond, refactor certain moves as required and implement all remaining multi hit moves except Dragon Darts.
The "x was hit with recoil!" message was appearing when the target was immune due to protection or an ability - this should fix that and handle any other cases where 0 damage is dealt.
Make these effects endure one hit of a multi strike move, not all of them.
Thief doesn't work as a move end effect - the item steal animation only works if it runs before the target faints. This restores its original code and adds a simple check to handle its interaction with Parental Bond.
Bug Bite's effect should only occur on the final strike of Parental Bond (which can also be the first strike if it KO's the target).
This removes the need to have separate move effects for Twineedle, Double Iron Bash, and any custom two strike moves.
Had a define in a comment, which offset all the new move effects and misaligned their battle scripts.
Removing move effects above the ones that have stat change animations causes the wrong stat change to be shown, so I've reordered the list to fix that.
Fixed it properly this time!
This is now checked in SetMoveEffect instead of in each move that should behave this way. Some effects will need to be redone to make them play nice with this change, but some just work as is.
Now works properly with and without Parental Bond. It wasn't broken before, but its interaction with Parental Bond was.
Only scatters coins on the first hit of Parental Bond.
Not sure if this should hit twice. If not, this works.
Dragon Tail, Natural Gift, Spit Up all hit twice now. Could maybe use isparentalbondlaststrike instead of some of the other checks too.
Bide no longer gets stuck in a loop, but also doesn't strike twice.
Add animation data provided by Mr. Dollsteak for all gen 1-6 Pokémon (barring megas).
ghoulslash and others added 18 commits November 4, 2022 09:29
Misc:
-Made the ability pop up stick around until the ability's effect ends just like in ORAS.
-Updated sText_BadDreamsDmg accordingly while I was at it.
Config for Dark Void being only usable by Darkrai
Doc missing uses of ST_OAM_OBJ_X in Pokémon Storage System
Fixed indentation in ScrCmd_faceplayer
…_20221030

# Conflicts:
#	src/pokemon_storage_system.c
…20221030

Support for Porymap 5.0.0 (pret merge 2022-10-30)
@AsparagusEduardo
Copy link
Collaborator Author

Set as draft until #2448 is fixed.

LOuroboros and others added 4 commits November 11, 2022 01:31
Issue #2448 
Haven't tested extensively, but regular Berries work and Bug Bite w/ Berries also works.
I believe this line was missing by accident (you can look at the merged Bug Bite PR #2433 to see that this line was deleted and never replaced).
Fix Berries that were broken after Bug Bite fix
@AsparagusEduardo AsparagusEduardo marked this pull request as ready for review November 12, 2022 21:23
@AsparagusEduardo
Copy link
Collaborator Author

Issue fixed, ready for merge.

@ghoulslash ghoulslash merged commit 23acbb3 into rh-hideout:master Nov 18, 2022
@AsparagusEduardo AsparagusEduardo deleted the RHH/pr/version/1.3.0 branch May 14, 2024 22:17
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.