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

Cata beta (draft) #2699

Closed
wants to merge 105 commits into from
Closed

Conversation

Hendo72
Copy link
Contributor

@Hendo72 Hendo72 commented Mar 24, 2024

No description provided.

- finished run-through of Orc starting area.
- modernized the code
- condensed steps where said steps shared a name and NPC.
- removed Level steps where not needed.
- shuffled steps for better class grouping.
- continue to modernize the code.
Completed run-through on this guide.
- Renamed 05_12_Bitsem_Durotar to Cata_INTRO_Orc_Troll_Part2
- Updated Guides.xml
- completed guide update from Sen'jin Village to Razor Hill.
- removed panda guides from .xml files
- brought my guides back into the restarted branch
- updated file name changes
- Added the missing `local guide =`
- Moved all of the archive files into it's own folder for convenience and to keep separate from the modified files.
- copied Horde Retail files into Cata folder
- Dropped the CATA_ from my INTRO files (not necessary)
- added Hakka's file changes to folder
- CATA_Guide_Hub still referenced in Alliance Guide.xml
- Forgot to drop CATA_ from my troll guide file name
- Removed Cata_Guide_Hub from Horde and updated .xml
I forgot to remove the BfA files.
Reset the Cata guides to using Retail header by copying over the Retail files into the Cata folder.

Intro guides were not touched.
- Added Z|1411;Durotar to Troll Intro
- removed all mentions of Chromie time  and updated next guide when it was Chromie Time
commit 0fb4982
Author: Ludovicus <ludovicus.maior@gmail.com>
Date:   Wed Mar 27 12:15:19 2024 -0400

    Add CATA starting zones.

commit 2e66f1c
Author: Ludovicus <ludovicus.maior@gmail.com>
Date:   Wed Mar 27 12:07:46 2024 -0400

    Add WoWPro.CATA

commit bc33c6f
Merge: e4eb117 f1ca7e5
Author: Ludovicus Maior <46032662+Ludovicus-Maior@users.noreply.github.com>
Date:   Wed Mar 27 10:35:06 2024 -0400

    Merge pull request Ludovicus-Maior#2698 from Cagomei/Z-tag-updates-23

    grizzly hills

commit f1ca7e5
Author: Cagomei <sshort8503@hotmail.com>
Date:   Sun Mar 24 00:17:27 2024 -0500

    grizzly hills
- Added GuideLevels to header as a workaround for the unpack error.
Appears Blizz decided they want to use MIDDLE instead of CENTER. Makes the error go away. We'll see if they change their minds again.
- made some step tweaks (C steps to R steps because they are really travel steps)
Not going to point fingers but I don't think it was a knife or fork that did it.
Added coding to adjust the Guide list to display correctly in Cata.
- adding lv markers for class quest additions
- added lv 3 class quests
- Removing Hot Rod (U|46856|) from |S| steps so hot key can be used for other steps.
Cata is not the same as CATA
Cata is not the same as CATA
Copy link
Contributor Author

@Hendo72 Hendo72 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's not what this commit is about.

This commit was about:

  • Added Cata checks same as GuideList
  • positioned white text above title bar inside frame window (I believe Spoony is tweaking this in Retail)

Hendo72 and others added 15 commits April 21, 2024 16:28
- Life of the Party requires you do your level 3 class first. Because of this, each class requires its own A step.
- added level checks and training steps for lv 4 and 5
- Files provided by Hakka.
Untested changes while servers are offline.
- Tol Barad added to ZoneData
-> I don't know what this means:
-- Collided {[2340]=true,[244]=true,count=2}
- coordinate and note tweaks
- Added lv 7 training
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change needs to be discarded. IDK how this happened.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which change are you referring to? Is it one line specific or the entire commit?

- added training and level check steps
@Hendo72
Copy link
Contributor Author

Hendo72 commented Apr 29, 2024

I'm closing this PR and starting a new one with less commits.

@Hendo72 Hendo72 closed this Apr 29, 2024
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.

None yet

2 participants