🐛 Abyssea onTrigger takes single KI and doesn't spawn NMs that requires 2+

[Hinekri]

• [x] I have paid attention to this example and will edit again if need be to not break the formatting, or I will be ignored
• [x] I have searched existing issues to see if the issue has already been opened, and I have checked the commit log to see if the issue has been resolved since my server was last updated
• [x] I have read and understood the Contributing Guide

Branch affected by issue

base

Steps to reproduce

Attempt to spawn NM in Abyssea that requires more than 1 KI, it takes the first KI in the list from the player and does not pop the NM.

This is all QMs in all zones, single KI NMs can be spawned.

Expected behavior

All required KI should be pulled at once and the NM should spawn.

[Hinekri]

On an unrelated note: If you click "new issue" from the issue page without being logged in and then use that popup to login... it sends you to a blank issue with no rules template to check off.

[TeoTwawki]

If you are in the reddit servers discord theres a related psa I posted about the url. Its a github problem we have control over

[Mortalelite]

Not sure if getting same error but this was tested with Briareus

Also note does remove first KI like mention and leaves the other 2

[claywar]

Yep. Its not fully implemented yet. Abyssea is completely use at your own risk. I only laid the foundation for future work.

[Mortalelite]

Um ok I tested this just removing the line 740 in abyssea.lua -- player:messageSpecial(ID.text.LOST_KEYITEM, keyItem) and Briareus spawned and deleted the 3 KI. Just wanted to share incase can get a fix going

[Hinekri]

Um ok I tested this just removing the line 740 in abyssea.lua -- player:messageSpecial(ID.text.LOST_KEYITEM, keyItem) and Briareus spawned and deleted the 3 KI. Just wanted to share incase can get a fix going

Good enough for my server ty.

[Chronos77]

Resolved with MR #3793