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

Update Eastern Plaguelands.lua #1868

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Machou
Copy link
Contributor

@Machou Machou commented Dec 21, 2024

Fix Location of Master Siegesmith Corvus in Eastern Plaguelands

Fix Location of Master Siegesmith Corvus in  Eastern Plaguelands
@NORPG
Copy link
Member

NORPG commented Dec 21, 2024

My guess is that some patch moved it to this location, but I'm not sure which one.

@Exodius
Copy link
Member

Exodius commented Dec 21, 2024

The coordinates in question (both the current in the code and the proposed change) are relative to the map from which you are looking at the NPC. If you are looking at DK starting area (Plaguelands: The Scarlet Enclave) you get 50.8, 30.2. If you are looking at Eastern Plaguelands you get 83.4, 46.1.
https://www.wowhead.com/npc=28500/master-siegesmith-corvus

I've been wanting to create a DK and re-do the quest chain for the lolz... Will check it out when I do.

@Exodius Exodius self-assigned this Dec 21, 2024
@Machou
Copy link
Contributor Author

Machou commented Dec 22, 2024

The coordinates in question (both the current in the code and the proposed change) are relative to the map from which you are looking at the NPC. If you are looking at DK starting area (Plaguelands: The Scarlet Enclave) you get 50.8, 30.2. If you are looking at Eastern Plaguelands you get 83.4, 46.1. wowhead.com/npc=28500/master-siegesmith-corvus

I've been wanting to create a DK and re-do the quest chain for the lolz... Will check it out when I do.

you can create a level 10 dk (orc magar for example) and go to Eastern Plaguelands, that's what I did, and since the coordinates on ATT weren't the right ones, hence my PR :)

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