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

Add support for multiple roamers #4762

Merged
merged 7 commits into from
Jun 15, 2024

Conversation

Bassoonian
Copy link
Collaborator

Heavily stripped down version of @Sneed69's RoamersPlus, preserving vanilla compatibility in its default state and simply making it far easier to add roamers beyond the initial Latis.

Due to the stripping down, I have opted not to depart from his branch and strip it down from there, as that would cause stripping down for expansion users who previously merged in his full branch and wish to retain its extra functionality.

People who collaborated with me in this PR

@Sneed69

Discord contact info

bassoonian

@AsparagusEduardo
Copy link
Collaborator

Did you test merging the original branch into a clean upcoming and then merging this PR?

@Bassoonian
Copy link
Collaborator Author

There’s going to be conflicts, but surely that’s unavoidable? I’d rather have a few super easy to solve conflicts than silently overwriting a bunch of features because I solved conflicts for them in a way they presumably don’t want

include/roamer.h Outdated Show resolved Hide resolved
src/roamer.c Outdated Show resolved Hide resolved
@AlexOn1ine
Copy link
Collaborator

AlexOn1ine commented Jun 15, 2024

I increased the roamer count to 10 and couldn't open the dex anymore.

/Edit it seems it cleared all dex flags. It worked after I reset them.

@AlexOn1ine AlexOn1ine merged commit 22994c7 into rh-hideout:upcoming Jun 15, 2024
1 check passed
AsparagusEduardo added a commit that referenced this pull request Jun 16, 2024
AsparagusEduardo added a commit that referenced this pull request Jun 16, 2024
kittenchilly added a commit to kittenchilly/pokeemerald-expansion that referenced this pull request Jun 17, 2024
@Bassoonian Bassoonian deleted the roamerrefactor branch July 27, 2024 19:15
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