Skip to content

Conversation

@Gescht
Copy link

@Gescht Gescht commented Nov 20, 2024

Rename in order to be properly recognized by the new turtle wow launcher

Gescht and others added 3 commits November 20, 2024 04:53
so that players dont look for him in The Crossroads
Better description for Regthar Deathgate
@Gahnfaust
Copy link

If i do this, then the Alliance and Horde Guides are missing a dependency. But if i dont do this the the whole addons doesnt work and i get error messages
Screenshot 2024-11-24 211354
Screenshot 2024-11-24 212202

@Gescht
Copy link
Author

Gescht commented Nov 28, 2024

If i do this, then the Alliance and Horde Guides are missing a dependency. But if i dont do this the the whole addons doesnt work and i get error messages Screenshot 2024-11-24 211354 Screenshot 2024-11-24 212202

Are you talking about the alliance and horde guides within this addon, or are you talking about a seperate addon entirely?
I cannot replicate the issues you are having.

@Gescht
Copy link
Author

Gescht commented Nov 28, 2024

I fixed the dependecy issues.
Still couldnt replicate the error messages you linked.

@Gescht
Copy link
Author

Gescht commented Nov 28, 2024

Okay, I kinda messed up by not seperating my changes into different branches for cleaner pull requests.

Changes made as of right now:

  • ranamed .toc file for launcher support

  • renamed dependencies of horde and alliance guides

  • small wording changes for cleaner instructions for Regthar (barrens)

  • several lua comments removed from various guides to fix those guides getting loaded (parser.lua:103 unknown action error)

  • removal of whitespaced that caused the same error

  • fix to display 49-50 feralas horde guide

@Gescht Gescht changed the title Rename TourGuideVanilla.toc to TourGuide-Turtle.toc added turtlewow launcher support and bugfix for guides not getting loaded Nov 28, 2024
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