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

feat(Core): add config option for LFG expansion #7256

Merged
merged 9 commits into from
Aug 30, 2021

Conversation

locus313
Copy link
Member

@locus313 locus313 commented Aug 7, 2021

Changes Proposed:

  • Add config option for dungeon finder expansion

Before this pr when you have tbc dungeons disabled a player see's this:

image

With this pr players will see this:
image

Issues Addressed:

  • Closes

SOURCE:

Tests Performed:

  • Build and run
  • set DungeonFinder.Expansion in your worldserver.conf
  • test in game with different config options to see how the client shows it to the player

How to Test the Changes:

  1. set DungeonFinder.Expansion = 0 in the worldserver.conf
  2. build and run
  3. set character level 59 (for tbc)
  4. Check in-game how the dungeon finder display's to the player

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.

@github-actions github-actions bot added the CORE label Aug 7, 2021
@FrancescoBorzi
Copy link
Contributor

nice one!

@locus313 locus313 marked this pull request as ready for review August 16, 2021 23:49
@FrancescoBorzi FrancescoBorzi merged commit 1c43e6a into azerothcore:master Aug 30, 2021
@locus313 locus313 deleted the rdf-expansion branch August 30, 2021 19:22
FrancescoBorzi added a commit to FrancescoBorzi/azerothcore-wotlk that referenced this pull request Oct 1, 2021
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