Skip to content
This repository has been archived by the owner on Dec 4, 2020. It is now read-only.

FixedPromyvions #458

Merged
merged 4 commits into from
Apr 9, 2020
Merged

Conversation

Solarsurge
Copy link
Contributor

*ALL Promyvion cutscenes now properly working according to mission progress.
*Fixed player dropoff after interacting with Shattered Telepoints during 2nd Promyvion
*Map acquisition now works properly in Holla and Mea

I affirm:

  • that I agree to Project Topaz's Limited Contributor License Agreement, as written on this date
  • that I've tested my code since the last commit in the PR, and will test after any later commits

*ALL Promyvion cutscenes now properly working according to mission progress.
*Fixed player dropoff after interacting with Shattered Telepoints during 2nd Promyvion
*Map acquisition now works properly in Holla and Mea
Copy link
Contributor

@ibm2431 ibm2431 left a comment

Choose a reason for hiding this comment

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

You got close to a no-comment approval, but were sabotaged by previous scripters! 😉

Comment on lines 16 to 17
npcUtil.giveKeyItem(player, tpz.ki.MAP_OF_PROMYVION_MEA)
player:confirmTrade()
Copy link
Contributor

Choose a reason for hiding this comment

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

I know you're just using what's currently in these files, but while we're here, we might as well fix a retail inaccuracy~!

These should be starting an event where the screen flashes and there's text about blinding white lights and images being burned into your mind. This is event 49 for all three promyvions.

(And then give the KI and confirm the trade in onEventFinish for those events)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

npcUtil.giveKeyItem(player, tpz.ki.MAP_OF_PROMYVION_MEA)
player:confirmTrade()
else
player:messageSpecial(ID.text.NOTHING_HAPPENS)
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this text display when you trade a bad item? Bad trades are usually just ignored.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

"Nothing happens..." is displayed on a bad trade. To my knowledge, that's correct.

@ibm2431 ibm2431 added the reviewed Has been reviewed by at least one Staff Member label Apr 7, 2020
*Added blinding light event on trade for map acquisition per request
*Fixed crash in Mea for missing text
@ibm2431 ibm2431 added merge ready reviewed and deemed ready to merge and removed reviewed Has been reviewed by at least one Staff Member labels Apr 8, 2020
Copy link
Contributor

@ibm2431 ibm2431 left a comment

Choose a reason for hiding this comment

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

Thanks Solar!

@zircon-tpl zircon-tpl merged commit f49b6cd into project-topaz:release Apr 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
merge ready reviewed and deemed ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants