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

Addon: [1.7.55] - Add Cataclysm Classic support #586

Merged
merged 5 commits into from
May 11, 2024
Merged

Conversation

Xian55
Copy link
Owner

@Xian55 Xian55 commented May 11, 2024

From Addon version 1.7.55 the addon recognizes the Cataclysm Classic client.
Added the necessary JSON files in order to make the the backend work with the new client.

The Creatures.json file is coming from Wrath, not yet updated with Cataclysm database, i have to find one. New NPC names were not showing up.

The backend were capable of recognizing the talent.json and talenttab.json files, however the links to wowhead seems to be broken, not a breaking change to prevent merging this feature.

Skinning supposed to work, however wowhead databases are still not yet full updated so in order to fill the empty data i've used retail dataset in that case.

exp.json needs to be validated for correct numbers.


I've tested the new client with a Level 20 Human Warrior at Loch Modan, which is an updated vanilla zone in Cata. I've used RemoteV3 mmaps and its generated a good looking path to the other side of the lake from Thelsamar(alli city hub).

LocalV1 and RemoteV1 pathfinder may only work at the zones which were not updated by the Cataclysm client.

However i suggest to use RemoteV3 for the time being!

Note: the provided mmaps files for Cataclysm client was not created by me. At first glance it seems fine, however it might not be suitable for player movement. Once TrinityCore team progresses forward and their extractor tool is ready to process Cataclysm Classic client, i'm going to revisit how to improve the navmesh for the project. Until then expect may more stuck situation during navigation.

@Xian55 Xian55 added documentation Improvements or additions to documentation feature request This is a request for a new feature, or an expansion of an existing feature. refactor This ticket concerns the possible simplification of code/data. enhancement This pull request implements a new feature. breaking change labels May 11, 2024
@Xian55 Xian55 linked an issue May 11, 2024 that may be closed by this pull request
@Xian55 Xian55 merged commit 2f150c9 into dev May 11, 2024
1 check passed
@Xian55 Xian55 deleted the exp/cata-classic-support branch May 11, 2024 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change documentation Improvements or additions to documentation enhancement This pull request implements a new feature. feature request This is a request for a new feature, or an expansion of an existing feature. refactor This ticket concerns the possible simplification of code/data.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for the Cataclysm Client
1 participant