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

merge #15

Merged
merged 11 commits into from
May 30, 2024
Merged

merge #15

merged 11 commits into from
May 30, 2024

Conversation

AthenaSui
Copy link
Owner

Changes Proposed:

This PR proposes changes to:

  • Core (units, players, creatures, game systems).
  • Scripts (bosses, spell scripts, creature scripts).
  • Database (SAI, creatures, etc).

Issues Addressed:

  • Closes

SOURCE:

The changes have been validated through:

  • Live research (checked on live servers, e.g Classic WotLK, Retail, etc.)
  • Sniffs (remember to share them with the open source community!)
  • Video evidence, knowledge databases or other public sources (e.g forums, Wowhead, etc.)
  • The changes promoted by this pull request come partially or entirely from another project (cherry-pick). Cherry-picks must be committed using the proper --author tag in order to be accepted, thus crediting the original authors, unless otherwise unable to be found

Tests Performed:

This PR has been:

  • Tested in-game by the author.
  • Tested in-game by other community members/someone else other than the author/has been live on production servers.
  • This pull request requires further testing and may have edge cases to be tested.

How to Test the Changes:

  • This pull request can be tested by following the reproduction steps provided in the linked issue
  • This pull request requires further testing. Provide steps to test your changes. If it requires any specific setup e.g multiple players please specify it as well.

Known Issues and TODO List:

  • [ ]
  • [ ]

How to Test AzerothCore PRs

When a PR is ready to be tested, it will be marked as [WAITING TO BE TESTED].

You can help by testing PRs and writing your feedback here on the PR's page on GitHub. Follow the instructions here:

http://www.azerothcore.org/wiki/How-to-test-a-PR

REMEMBER: when testing a PR that changes something generic (i.e. a part of code that handles more than one specific thing), the tester should not only check that the PR does its job (e.g. fixing spell XXX) but especially check that the PR does not cause any regression (i.e. introducing new bugs).

For example: if a PR fixes spell X by changing a part of code that handles spells X, Y, and Z, we should not only test X, but we should test Y and Z as well.

trickerer added 11 commits May 25, 2024 22:37
(cherry picked from commit 642b22fb7a2c4aea5a3ce9b20765e4318f2195ce)
(cherry picked from commit 6148e797cf149e5a894578970fb214977570cfb3)

# Conflicts:
#	data/sql/custom/db_characters/2024_05_29_00_characters_npcbot_logs.sql
#	data/sql/custom/db_world/2024_05_29_00_command.sql
#	src/server/game/AI/NpcBots/botdatamgr.cpp
(cherry picked from commit 2e6e2d8e8bcbca534cd937ce9f5ad80905829697)
…use eventmap instead

(cherry picked from commit 777c8832d3b12ead931292bafc4e77881de7c863)
…gging

(cherry picked from commit df0d0cb2b4c77e786ee956e9027fc6c3450d1fc3)

# Conflicts:
#	src/server/worldserver/worldserver.conf.dist
…` log

(cherry picked from commit 860433723b7439d9a01389b57870558681bbd5c9)

# Conflicts:
#	src/server/game/AI/NpcBots/botmgr.cpp
(cherry picked from commit b421138ccbd4f6c6ca786f86f8dd02902aa561a7)

# Conflicts:
#	src/server/game/AI/NpcBots/botmgr.cpp
…a different map id from bot's spawn map

(cherry picked from commit e43c1e9969051a27ba2c4dbe05641c45a14dba0c)
@AthenaSui AthenaSui merged commit 3680c83 into AthenaSui:Az_trickerer_npcbot May 30, 2024
10 checks passed
AthenaSui pushed a commit that referenced this pull request Jul 30, 2024
Playerbot fixes and functionnality to talk in channels
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.

2 participants