add StartQuest ipc #133

Merged
liza merged 2 commits from croizat/Questionable:ipc into master 2025-02-05 17:02:54 +00:00
Contributor
No description provided.
croizat added 1 commit 2025-02-05 15:59:48 +00:00
croizat added 1 commit 2025-02-05 16:14:50 +00:00
liza reviewed 2025-02-05 16:24:55 +00:00
@ -38,1 +44,4 @@
private static bool StartQuest(QuestController qc, QuestRegistry qr, string questId)
{
if (ElementId.TryFromString(questId, out var elementId) && elementId != null && qr.TryGetQuest(elementId, out var quest))
Owner

You can use qr.IsKnownQuest(elementId) instead of qr.TryGetQuest(elementId, out var quest)

You can use `qr.IsKnownQuest(elementId)` instead of `qr.TryGetQuest(elementId, out var quest)`
Author
Contributor

Well I need the out value to pass to SetNextQuest, no? Unless there's a method I'm missing that would only take in an elementid

Well I need the `out` value to pass to `SetNextQuest`, no? Unless there's a method I'm missing that would only take in an elementid
Owner

Oh, you're right.

Oh, you're right.
liza marked this conversation as resolved
liza merged commit 7a47992792 into master 2025-02-05 17:02:54 +00:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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#133
No description provided.