liza/Questionable
liza/Questionable
2
1
Fork 18
Code Pull Requests 1 Releases 84 Activity

[ShB][Allied Society][Pixies] Added remaining story quests #45

Merged
liza merged 3 commits from plogon_enjoyer/Questionable:pixie into master 2024-09-13 07:09:33 +00:00
Conversation 0 Commits 3 Files Changed 2 +276
plogon_enjoyer commented 2024-09-12 17:22:19 +00:00
Contributor
Copy Link

Added the following story quests:

  • As the Heart Bids
  • Forever and a Dream
Added the following story quests: - As the Heart Bids - Forever and a Dream
plogon_enjoyer added 2 commits 2024-09-12 17:22:19 +00:00
liza reviewed 2024-09-12 17:30:34 +00:00
QuestPaths/5.x - Shadowbringers/Allied Societies/Pixies/Story/3687_As the Heart Bids.json
@ -0,0 +152,4 @@
},
"TerritoryId": 816,
"InteractionType": "SinglePlayerDuty",
"$": "Sequence 7 is the duty itself"
liza commented 2024-09-12 17:30:34 +00:00
Owner
Copy Link

This looks like it's wrong. Quest sequences have nothing to do with instanced battles. It's more that Sequence 7 is probably cutscene-only, and thus will likely never be seen unless you (I think) DC in the middle of it.

Quite contrary: if you look at e.g. The Mountain that Strides, you'll see sequence 6 is a SinglePlayerDuty and sequence 7 is some other interaction afterwards.

For all intents and purposes, this should have

{
 "Sequence": 7
}

as separate sequence.

This looks like it's wrong. Quest sequences have nothing to do with instanced battles. It's more that Sequence 7 is probably cutscene-only, and thus will likely never be seen unless you (I think) DC in the middle of it. Quite contrary: if you look at e.g. `The Mountain that Strides`, you'll see sequence 6 is a SinglePlayerDuty and sequence 7 is some other interaction afterwards. For all intents and purposes, this should have ``` { "Sequence": 7 } ``` as separate sequence.
plogon_enjoyer commented 2024-09-13 06:45:22 +00:00
Author
Contributor
Copy Link

Ohhh good point, as I was also running TextAdvance so the cutscene was naturally skipped. I'll update this now.

Ohhh good point, as I was also running `TextAdvance` so the cutscene was naturally skipped. I'll update this now.
plogon_enjoyer commented 2024-09-13 06:49:42 +00:00
Author
Contributor
Copy Link

This is now updated in 6f8693340c

This is now updated in 6f8693340ce35e51c52a7ddd2c6bfd2199ca39cc
plogon_enjoyer marked this conversation as resolved
plogon_enjoyer added 1 commit 2024-09-13 06:49:16 +00:00
liza merged commit 14a69446fa into master 2024-09-13 07:09:33 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: liza/Questionable#45
No description provided.
Delete Branch "plogon_enjoyer/Questionable:pixie"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?