Skip to content

Merged data from mob_race2_db to mob_db#3394

Merged
hemagx merged 3 commits intoHerculesWS:masterfrom
MrKeiKun:refactor-mob_race2_db
Apr 21, 2026
Merged

Merged data from mob_race2_db to mob_db#3394
hemagx merged 3 commits intoHerculesWS:masterfrom
MrKeiKun:refactor-mob_race2_db

Conversation

@MrKeiKun
Copy link
Copy Markdown
Contributor

@MrKeiKun MrKeiKun commented Sep 6, 2025

Pull Request Prelude

Changes Proposed

Merged data from mob_race_db.txt on both renewal and pre-renewal inside their respective mob_db.conf

Issues addressed: #3127

Signed-off-by: Lorenzo Buitizon <the.keikun@gmail.com>
@MrKeiKun
Copy link
Copy Markdown
Contributor Author

MrKeiKun commented Sep 6, 2025

I noticed that ASSULTER is RC2_NINJA and RC2_TURTLE does it mean a monster can have two RaceGroup?

Copy link
Copy Markdown
Contributor

@Asheraf Asheraf left a comment

Choose a reason for hiding this comment

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

Please add removal notice error similar to what we have for mob_avail.txt

@MrKeiKun MrKeiKun requested a review from Asheraf September 9, 2025 12:23
Comment thread src/map/mob.c Outdated
…s deprecation and merger into mob_db.

Signed-off-by: Lorenzo Buitizon <the.keikun@gmail.com>
@MrKeiKun MrKeiKun force-pushed the refactor-mob_race2_db branch from cf9c0cc to 2ad40bb Compare September 9, 2025 16:21
@hemagx hemagx added this to the Release v2026.04 milestone Apr 19, 2026
@hemagx
Copy link
Copy Markdown
Contributor

hemagx commented Apr 19, 2026

@MrKeiKun While official race2 is quite small and easy for hand migration, custom ones might not be, so we will also require a conversion script in place to move forward with this PR.

@MrKeiKun
Copy link
Copy Markdown
Contributor Author

MrKeiKun commented Apr 19, 2026

I will PR later the script 😘

Update:
Done with the script.

@hemagx hemagx merged commit 64ad853 into HerculesWS:master Apr 21, 2026
14 of 56 checks passed
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