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

Fix "A Thief in Norg?!" content group IDs #5689

Merged
merged 1 commit into from
May 11, 2024
Merged

Conversation

claywar
Copy link
Contributor

@claywar claywar commented May 11, 2024

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

  • Corrects all mob spawn positions for A Thief in Norg BCNM based on retail captures
  • Fixes the required group to not include Onki's elemental
  • Corrects default actions for SAM quests leading up to this BCNM
  • Corrects missing variable for entry requirement
  • Adds jobspecial mixin for required enemies
  • Adds missing entries for Ayakashi in SQL so that it will spawn correctly on Astral Flow

Steps to test these changes

Run through this battlefield in all areas. Note: Additional issue raised to support the second avatar that uses Astral Flow in this BCNM.

@claywar claywar force-pushed the bugfix/thief-in-norg-moblist branch 2 times, most recently from 0999914 to e917d32 Compare May 11, 2024 19:10
@claywar
Copy link
Contributor Author

claywar commented May 11, 2024

Good to go now, Ayakashi will spawn appropriately, though until a separate PR goes in, if GM is enabled, you won't get the ability triggered.

(vanadielHour > 18 or vanadielHour < 5) and
not GetMobByID(valkurmID.mob.DOMAN):isSpawned() and
not GetMobByID(valkurmID.mob.ONRYO):isSpawned()
(vanadielHour > 18 or vanadielHour < 5)
Copy link
Member

Choose a reason for hiding this comment

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

Is this supposed to be 19 to 4 or 18 through 5? (>= and <= or > and <)

Copy link
Contributor Author

@claywar claywar May 11, 2024

Choose a reason for hiding this comment

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

Definitely should be >=/<, good catch

@claywar claywar force-pushed the bugfix/thief-in-norg-moblist branch 2 times, most recently from 0235dac to 576fe5b Compare May 11, 2024 19:17
Turn off pet spawn, since mob_pets spawn automatically

Add missing variable to getVar

Fix default actions for SAM AF2, AF3

Correct spawn points, add additional NPCs for Waughroon Shrine

Add jobspecial mixin for enemies

Add missing mob entries, cleanup groups

Remove avatars from required group

Fix conditional for Yomi Okuri timing

Fix conditional again
@claywar claywar force-pushed the bugfix/thief-in-norg-moblist branch from 576fe5b to 4d3da59 Compare May 11, 2024 19:38
@claywar claywar merged commit 8c6e3ad into base May 11, 2024
11 checks passed
@claywar claywar deleted the bugfix/thief-in-norg-moblist branch May 11, 2024 20:08
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

3 participants