Warcraft-GoA-Development-Team / Warcraft-Guardians-of-Azeroth-2

"Guardians of Azeroth" is a mod for Crusader Kings 3 with a worldwide development team, based on the Warcraft universe by Blizzard Entertainment. In this mod, our team is working to recreate the world of Azeroth in all of its glory, particularly focusing on the story of the classic Warcraft strategy games and World of Warcraft Classic.
79 stars 59 forks source link

1.12.5 update #1391

Closed edmund-h closed 1 month ago

edmund-h commented 1 month ago

Changelog (from 1.12 fast follow):

Completion Tracker:

Modified files - use this page to find which files need updating

VS Code native procedure for checking modified files used by :

  1. add your name next to the folder you are working on
  2. go to the page listed above and locate a file in the folder you are working on in the modified files (white text) section of the CK3 Game Content box
  3. if you do not already have it, add the game files into your VS Code project
  4. locate and right click the file in game files, select Select for Compare
  5. if there is a corresponding file in the GoA2 files, right click it and select Compare with Selected (otherwise move on to the next file
  6. reconcile the game file on the left (deleted text in red) to the GoA2 file on the right (added text in green)
    • if added text does not have a #Warcraft comment, you need to check Git Blame to see if we added it. If we did not, make an assessment as to whether the deleted content serves the mod. If you leave it in, leave a comment stating it was removed in 1.12.x in case it causes a problem later. Otherwise, if we did add it, add a #Warcraft comment
    • "removed" text is not necessarily deleted by us. new material in the file will show up as "removed". generally, you should just use the arrow button to immediately add it, especially if it is related to a new feature. However, it is also a good idea to check the git blame if you are in doubt to see if and when we removed the text. If you do not add the "removed" content in, add a # warcraft comment with a brief reason as to why. If someone else removed the content and did not leave that comment, then you should add it.
    • feel free to ask in development chat if you are in doubt

Common

Events

GFX

GUI

History

Localization

Tests:

How to test:

Raged50 commented 1 month ago

checked errorlog and playtest for 20 mins

nothing in errorlog relevant to this update and all graphical issues (nerubians) working, and forsaken event is tested and works fine

requires 1-2 more reviews please

helloruhe commented 1 month ago

fixed minor stuff in errorlog (that didnt really matter but might as well anwyays) LGTM