Converted RoE to quest framework#10731
Open
greentchristian-LSB wants to merge 2 commits into
Open
Conversation
Contributor
Author
|
If Lamepaue needs to be excluded from this PR because of the check, that's fine; as mentioned above, it was only included because she came up while I was searching RECORDS_OF_EMINENCE, and the framework for rewatching the CS was already present. The only changes I made to her script was to uncomment the lines related to RoE, filled in the placeholder values, and added a single line to make sure that rewatching the RoE CS would not immediately play Fetichism afterward since the RoE CS itself supplies option 1 when watching all the way through. |
Reverts Lamepaue since uncommented code isn't passing automated checks upstream.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I affirm:
What does this pull request do?
Moves the start of Records of Eminence out of NPC scripts into its own quest script, making it show up in the quest log as in retail. Detects if you have started the tutorial, playing the correct version of the CS. Removes a line that was present in each of the RoE NPCs that duplicated the reward message upon completing the quest. And enabled rewatching the CS with the Bastok Markets Melody Minstrel since the framework was already in place - building the other three Melody Minstrels scripts from scratch would probably count as scope creep for this fix.
Steps to test these changes
Local tests: Create a new character in any nation. Try talking to the RoE NPC to have them tell you to set First Step Forward. After setting First Step Forward, talk to the RoE NPC again and get the CS. Verified that selecting 'Walk Away' does not complete the quest on all 4 NPCs. Verified that you get the correct version of the CS depending on if you have spoken to Gulldago\Alaune\Selele prior to starting the CS. Eternal Flame only has one version of the CS. Verified that expanded RoE objectives are available after completing the CS. Quest appears under Completed -> Other Areas -> Records of Eminence upon completion. Sparks shop available upon completion. NPCs accept vouchers as trades, ensured they do not accept random junk.
Retail tests: Checked that simply having talked to Gulldago\Alaune\Selele is sufficient for shorter CS to play. Checked with Melody Minstrels and learned that replaying CS always shows the extended version, regardless of tutorial completion.