Azeroth-Pilot-Reloaded / azeroth-pilot-reloaded

Azeroth Pilot Reloaded
GNU General Public License v3.0
29 stars 10 forks source link

[ISSUE] - Quest names not showing up #143

Closed burntorangejedi closed 1 year ago

burntorangejedi commented 1 year ago

Description

When APR opens, there is a lot of stuff for me to do, but none of the quest names are showing up. Is there a dependency I'm missing somewhere?

APR Version

APR 2.3.4

APR Route

Speedrun

World of Warcraft Flavor

Retail (Default)

Tested with only APR

Lua Error

None

Reproduction Steps

  1. Log in to character in Burning Crusade on Chromie Time
  2. Observe

Last Good Version

No previous version tried

Screenshots

image

Any additional information?

No response

Pahonix commented 1 year ago

The quest names are not available to the addon until you have seen them the first time. On subsequent play throughs, the addon will have cached the quest names and will be able to display them in the QuestOrderList.

burntorangejedi commented 1 year ago

Thanks for answering my question, but perhaps, as a feature recommendation, you could add Grail support so that the names are available from the start?

NightofStarrs commented 1 year ago

This is something that can be looked at in the future (possible for version 3). The issue was, we did not want to have a list of quest names to iterate over, thus the addon runs solely on the QuestID of the quest (or intractable object), and then the quest parts/path. This was done originally to save space, and processing power, but the addon is written to display the QuestID and Quest Name once it has been set up in your WoW Cache folder, thus relying on you having previously interacted with the quest at some time in the past (on a previous character).

APR (and the predecessor) AAP were designed for Alt Leveling, by taking the guess work out of leveling additional characters, not for a first time play through.

NightofStarrs commented 1 year ago

We are going to look into using Grail, but we need to evaluate the addon, and see if we can hook into it, or if we need to have it side-loaded with APR. I will go ahead and close this for now, and instead change it into a Feature Request instead of an Issue.