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

Fixes text output formatting due to core change #57

Merged
merged 1 commit into from
Mar 19, 2025

Conversation

ghost
Copy link

@ghost ghost commented Mar 18, 2025

Changes Proposed:

Issues Addressed:

SOURCE:

#48 (comment)

Tests Performed:

  • Recompiled and tested in game by visiting various dungeons
  • Tested dungeons much lower level, similar level, and higher level
  • Verified correct output text to chat window.

How to Test the Changes:

  1. Follow my tests performed above

Additional credit: https://github.com/Povotech (#48 (comment)) as I used their example as the starting point.

- Fixes module output text formatting due to core change.

<!-- If your fix has a relating issue, link it below -->
- Closes 48

<!-- If you can, include a source that can strengthen your claim -->
#48 (comment)

<!-- Does it build without errors? Did you test in-game? What did you test? On which OS did you test? Describe any other tests performed -->
- Recompiled
- Tested various dungeons at, below, and above my characters level
- Noted proper output

<!-- Describe in a detailed step-by-step order how to test the changes -->

1.create a level 40 character
2.visit low, mid and high level dungeons
3.verify proper output
@Helias
Copy link
Member

Helias commented Mar 19, 2025

thanks for your PR!

@Helias Helias merged commit 7540918 into azerothcore:master Mar 19, 2025
1 check 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.

Bug: Message enter instance bug variable
1 participant