hythloday / VenturePlanSoDMissions

Addon to bring VenturePlan up to date with the 9.1 missions
GNU General Public License v3.0
134 stars 36 forks source link

9.2 changes #69

Closed zealvurte closed 1 year ago

zealvurte commented 2 years ago

9.2 only needs some new spells.

gaveer commented 2 years ago

@zealvurte in which file i add these changes ? in extra-vs-spells file there is 2 section : T.MoreSpells and T.UnverifiedSpells

zealvurte commented 2 years ago

T.MoreSpells afaik.

hythloday commented 2 years ago

Thanks for this, I'm not playing at the moment but I'll get this merged when I have a moment.

@zealvurte could you message me on Discord? ItsJames#0297

solkarnar commented 2 years ago

Just dropping in to say thanks to everyone working on this!

Johnydoe1977 commented 2 years ago

Hi, has the new 9.2 missions been included?

woefulwabbit commented 2 years ago

@hythloday You'll want to merge from https://github.com/Divergentcurl/VenturePlanSoDMissions/tree/cumulative-fixes too

tgeisterfer commented 2 years ago

i havent updated ventureplan in ages. where do i start with all the new updates

glassleo commented 2 years ago

i havent updated ventureplan in ages. where do i start with all the new updates

Yeah this is very all over the place and disorganized.

greghudson commented 2 years ago

[352]={type="aura", modDamageTaken=30, target=4}, [353]={type="aura", modDamageDealt=20, target=1},

I think these need a duration=; otherwise I get lua errors predicting Automa Attack. I'm guessing the value should be 2 based on in-game tooltips, but I'm less certain about that.

zealvurte commented 2 years ago

Ah, I forgot I had to modify VP for those 2 special cases. Simulations showed these auras with no duration are never cast, so I've replaced them with nop for VP.

Divergentcurl commented 2 years ago

i havent updated ventureplan in ages. where do i start with all the new updates

Yeah this is very all over the place and disorganized.

I was frustrated by this as well. That is why I have gathered all the fixes I know in my cumulative fixes branch. If anyone notices anything missing, please let me know and I will add it.

Briandris commented 2 years ago

I'm confused and fairly new to Github. I really appreciate those of you who are keeping this addon functional for people like me who might as well be reading a foreign language.

Previously, I followed instructions for "Getting it Running" and "Hotfixing the Code" for extra companions. I installed hythloday's Ventureplansodmissions. It was working great until 9.2 hit this week.

I don't know how to use/install the cumulative fixes branch. There's nothing to download or copy like the "latest release" button for Hythloday's page.

I did see the added change for overideAA and put that in the ventureplan file. I deleted the old VentureplanSoDMissions and did a fresh download and put it in the addons file I copy/pasted the new spells kindly posted from @zealvurte.

When I load WoW and the open the adventure table, there are alerts on the missions Prison Break, Gatecrashers, Customary Shakedown, Defeat the World Eaters and Piratical Problems. Could someone help me understand where I likely went wrong please?

glassleo commented 2 years ago

I don't know how to use/install the cumulative fixes branch. There's nothing to download or copy like the "latest release" button for Hythloday's page.

Go to Divergentcurl's repo page for the branch and click the green "Code" button in the top right corner and select "Download ZIP".

Note that the folder needs to be renamed to VenturePlanSoDMissions for the game to load it properly. (It will be named VenturePlanSoDMissions-cumulative-fixes in the ZIP archive).

Briandris commented 2 years ago

@glassleo Fixed! Thanks so much!

siweia commented 2 years ago

For mission 2323, https://www.wowhead.com/mission=2323/joint-operations-the-hunt, there should be four rewards include follower XP. VP only displayed three reward icons and anchoring weried. 112

solkarnar commented 2 years ago

For mission 2323, https://www.wowhead.com/mission=2323/joint-operations-the-hunt, there should be four rewards include follower XP. VP only displayed three reward icons and anchoring weried. 112

Seeing that this mission requires 5 icons (1 exp + 4 rep), could it be due to an icon overflow problem? @zealvurte

zealvurte commented 2 years ago

Almost certainly. I noticed it, but didn't look into resolving it at the time or since, as it's a one off so far.

zealvurte commented 2 years ago

There's been a hotfix to fix (most of) the issues I highlighted in the data. I've updated the original post with the changes.

Divergentcurl commented 2 years ago

@zealvurte Thanks for the swift update. I have updated my branch as well.

LostTemple1990 commented 2 years ago

sim error results and real logs.txt enemy in slot 5 cast spell 363 to itself in turn 13 the value of key "target" to spell 363 may be friend-front-soft but not friend-front-hard

zealvurte commented 2 years ago

Verified and corrected, thanks - my logs clearly didn't encounter a case where it made a difference.

Also taking another look at my logs for spell 46, I have none that verify a difference between hard and soft either, so I've unverified that.

Barleduq commented 2 years ago

I don't know how to use/install the cumulative fixes branch. There's nothing to download or copy like the "latest release" button for Hythloday's page.

Go to Divergentcurl's repo page for the branch and click the green "Code" button in the top right corner and select "Download ZIP".

Note that the folder needs to be renamed to VenturePlanSoDMissions for the game to load it properly. (It will be named VenturePlanSoDMissions-cumulative-fixes in the ZIP archive).

Ok, so the zip file from the cumulative-fixes branch should replace the current version of VPSODM 0.10 ? Or is this more files that I have to insert in places? (I'm comfortable modifying files, but I don't always understand what's going on with repos and commits - my coding days were all in college, way before version control was a thing.)

glassleo commented 2 years ago

Ok, so the zip file from the cumulative-fixes branch should replace the current version of VPSODM 0.10 ? Or is this more files that I have to insert in places? (I'm comfortable modifying files, but I don't always understand what's going on with repos and commits - my coding days were all in college, way before version control was a thing.)

I'm assuming you managed to download the ZIP file correctly, and also followed the instructions for modifying the VenturePlan addon already.

  1. Open your WoW AddOns folder (usually C:\Program Files (x86)\World of Warcraft\_retail_\Interface\AddOns if you are running Windows)
  2. Delete the folder named VenturePlanSoDMissions inside the AddOns folder if it exists
  3. Open up the ZIP file you downloaded from Diventcurl's GitHub
  4. Inside the ZIP, there will be a folder named VenturePlanSoDMissions-cumulative-fixes, drag this into your AddOns folder to extract it, then close the ZIP file
  5. Rename the folder VenturePlanSoDMissions-cumulative-fixes inside your AddOns folder to VenturePlanSoDMissions
  6. That's it
Barleduq commented 2 years ago

Yes, I've modified the Widgets file when that fix came out in August, and the vs.lua file just now. I downloaded VPSoDM from wowinterface.com. That folder only has 2 files in it, and the cum fixes folder has 2 folders and 3 files more. So I was making sure I was doing the correct thing with it. I hope I can hear about those fixes being included in the main file (and, presumably uploaded to WoWI) so I can continue my usual addon installation habits.

Thank you for the fast reply! -Barleduq

Jevnaker commented 2 years ago

I tried to use this and now this addon (the ventureplan modifier one) thinks i havent modified the ventureplan files required.

Divergentcurl commented 2 years ago

Please have a look at https://github.com/Divergentcurl/VenturePlanSoDMissions/tree/cumulative-fixes#getting-it-running. It should help you get these fixes running. You only have to do this once.

Jevnaker commented 2 years ago

ok ty

Jevnaker commented 2 years ago

the issue i have is that i had already done that before implementing this fix thingy in this thread and now it believes i havent done it despite the correct things have been changed like in the link you shared with me.

Divergentcurl commented 2 years ago

I don't know how to help you further with the information you have given me. Maybe you could make a backup of both addons and try again with a fresh install? If that doesn't work out you can always restore the backups.

teelolws commented 2 years ago

Mikanikos was "nerfed" in 9.2.5. Bugfix to match its actual tooltip. Spell 85 modDamageTaken from -5000 to -50

solkarnar commented 2 years ago

Mikanikos was "nerfed" in 9.2.5. Bugfix to match its actual tooltip. Spell 85 modDamageTaken from -5000 to -50

[85]={type="aura", target=3, duration=2, firstTurn=3, modDamageTaken=-50},

This should be the new value in the vs-spells.lua file right?

zeenk commented 2 years ago

[85]={type="aura", target=3, duration=2, firstTurn=3, modDamageTaken=-50},

This should be the new value in the vs-spells.lua file right?

Yes

woefulwabbit commented 2 years ago

@hythloday Are you able to add a few collaborators to the project e.g. @Divergentcurl @zealvurte so they can merge updates and create releases?

Barleduq commented 2 years ago

Adding to the request at least at the point of creating releases such that the CurseBreaker addon can pick it up automagically. (Currently hitting 'code' button for more recent .zip file on Divergentcurl's fork, and installing it as instructed above, and hoping I'm doing all that needs doing.)

Divergentcurl commented 2 years ago

Thanks for reporting the Mikanikos change. I have updated my branch and have created a new project on WOWInterface to make it easier for everyone to update.

Barleduq commented 2 years ago

Wow, thanks! I've made a comment there for clarification, in case anyone just comes across it there and is confused.

rewzone commented 2 years ago

Please have a look at https://github.com/Divergentcurl/VenturePlanSoDMissions/tree/cumulative-fixes#getting-it-running. It should help you get these fixes running. You only have to do this once.

I'm having an issue following your added instructions around overrideAA. That code doesn't seem to be in the vs.lua file for me. I also noticed I'm running 4.22 which is the current version of Venture Plan on Curse.

greghudson commented 2 years ago

With Venture Plan 4.22 you shouldn't need the fixes from this addon any longer. (At least in my experience; I haven't failed a mission since I updated to 4.22.)