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
[Mission] Sandy 6-2 Adds table and npcUtil.popFromQM & shift proof popped NM #5633
base: base
Are you sure you want to change the base?
Conversation
fcb7270
to
9a1ae21
Compare
} | ||
mission.reward = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpick: can you add a space between these :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Anything for you my friend.
mission:setVar(player, 'Option', 1) | ||
else | ||
player:setMissionStatus(mission.areaId, 5) | ||
end | ||
end, | ||
|
||
[425] = function(player, csid, option, npc) | ||
if npcUtil.giveItem(player, { { xi.item.PICKAXE, 3 } }) then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are this being reverted? Wasnt there a PR precisely for this to be set at 3?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because I suck at github.
@@ -16,6 +16,13 @@ local krtID = zones[xi.zone.KING_RANPERRES_TOMB] | |||
|
|||
local mission = Mission:new(xi.mission.log_id.SANDORIA, xi.mission.id.sandoria.RANPERRES_FINAL_REST) | |||
|
|||
local ranparesRestTable = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spelling, ranperres
not GetMobByID(krtID.mob.CORRUPTED_YORGOS):isSpawned() and | ||
not GetMobByID(krtID.mob.CORRUPTED_SOFFEIL):isSpawned() and | ||
not GetMobByID(krtID.mob.CORRUPTED_ULBRIG):isSpawned() | ||
npcUtil.popFromQM(player, npc, ranparesRestTable, { claim = true, hide = 0, }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this have an impact if sneak pulling? Which is possible here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point - removing the claim = true should negate that.
@@ -2,4 +2,5 @@ local ID = zones[xi.zone.KING_RANPERRES_TOMB] | |||
|
|||
return { | |||
['_5a0'] = { messageSpecial = ID.text.HEAVY_DOOR }, | |||
['Tombstone'] = { messageSpecial = ID.text.A_TOMB }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please align these
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.,
[Mission] Sandy 6-2 [Mission] Sandy 6-2 [Mission] Sandy 6-2 Adds table [Mission] Sandy 6-2 table, npcUtil default diaglog [Mission] Sandy 6-2 Adds table and npcUtil.popFromQM & shift proof popped NM [Mission] Sandy 6-2 Adds table and npcUtil.popFromQM & shift proof popped NM [Mission]Ranperres_Final_Rest added table for mobs/shift proff Co-Authored-By: Frank <105882754+Frankie-hz@users.noreply.github.com> Co-Authored-By: Abdiah <62350957+dallano@users.noreply.github.com>
I affirm:
What does this pull request do?
Makes Sandy mission 6-2 shift proof for IDs (GetFirstID) for each of the 3 spawned mobs in KRT.
Adds table of mobs and moves to npcUtil.popFromQM
Steps to test these changes
!addmission 0 17
!setmissionstatus [NAME] 1 0
!pos -39 4.823 20 190
Click on door and FIGHT the mobs (!hp 0 will not progress missionstatus Var.
After death click door, will then ask to enter door. Select yes and then click the tombstone for KI Ancient Sandorian Book