LiteMount

LiteMount

2M Downloads

TWW - Adjust for skyriding/steady flight

mbattersby opened this issue ยท 14 comments

commented

Probably a lot of stuff that needs looking at including

  • flags and UI for flags
  • default SHIFT behaviour for downgrading mount type
  • conditions for which flight type is active
  • actions for changing flight type (maybe) "SwitchFlightStyle"
  • default behaviour depending on flight type not required
  • detecting (if any) mounts that are only steady flying that don't work if skyriding is on done by mountTypeID
commented

The new continent has steady flight disabled until you unlock it somerhow, I think at max level. Seriously Blizzard?

Starting out, IsAdvancedFlyableArea() == true and IsFlyableArea() == false, which I assume will switch to true, true once you unlock.

Not sure what happens if you switch to Steady Flight without it unlocked though. Edit: you can't switch before unlocking.

commented

There is a buff for Flight Style: Skyriding and a buff for Flight Style: Steady Flight but they are only present in areas you can switch between the two. Khaz Algar before unlocking Steady Flight has no buff present.

commented

There is an unlock spell for Khaz Algar Skyriding that you get from an early quest

https://www.wowhead.com/beta/spell=459188
https://www.wowhead.com/beta/quest=78533

Neither IsSpellKnown or IsPlayerSpell return true for it though and it doesn't look like it does anything (at the same time the buff "Dangerous Skies" that prevents skyriding goes away). It's possible it teaches you Dragonriding if you never knew it before, but since there is now C_MountJournal.IsDragonridingUnlocked() there's no need to look at it. I hope.

commented

I don't even want to think about making Tindral Sageswift work when people go back there >_<

commented

Blue post says (below) which I should check.

The ground mounts in your collection will continue to stay grounded, but most flying mounts will be able to be put to use for Skyriding save for a select few such as the Otterworldly Ottuk Carrier, some fish mounts, and brooms.

commented

This is what it looks like before it is completed
image

commented

Unfortunately it hasn't awarded the exploration achievement retroactively so it will probably take levelling a brand new character to collect them. I don't have the time to do that right now.

commented

No problem at all! I don't have the time either, I just thought I would check in case.

commented

Thanks, have you completed this? Is it account wide? Can you send me the output of /dump GetAchievementInfo(40231)?

commented

Does IsFlyableArea() start returning true in Khaz Algar after you complete it?

commented

Note to self: don't break classic.

commented

It seems ok for now, I think later I can probably remove DRAGONRIDING if there are no mounts that can't only steady fly left, or I handle them another way.

commented

I'm gonna leave DRAGONRIDING in, because people could well want different mounts for it versus FLY.