Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NPC Dialogue Update 1.2: job-specific background "survivor stories" #26932

Merged
merged 5 commits into from
Dec 15, 2018

Conversation

I-am-Erk
Copy link
Member

@I-am-Erk I-am-Erk commented Dec 3, 2018

Summary

SUMMARY: Content "NPC Dialogue: role-specific survivor stories"

Purpose of change

Creates survival stories that are specific to survivors that had certain jobs or roles at the time of the cataclysm. In specific this applies to teenagers, criminals, and numerous professionals.

Several of these stories are incomplete, hence this being marked as a WIP.

Describe the solution

These use traits that are assigned based on NPC class - see #26929

These also use snippets I have added in another PR - #26930

These options should only be visible in NPCs that are part of your party. Adding the first dialogue option to the NPC seems to be hard-coded: I may need some help getting that started, although since I created the initial pull request I've learned a lot.

Describe alternatives you've considered

Not applicable.

Additional context

Previously part of another pull request, #26834. This file has not been changed since that PR. Split up for ease of review. And that's the last of it... off to close that PR now.

progress

  • Create "table of contents" with all traits linked to talk_friend
  • Complete writing of all stories - change of plans, I will do these in smaller individual PRs to simplify review. Some of these stories are stubs, but it gets the point across.
  • Ensure appropriate cross-connectivity of all dialogue
  • Test in-game

talk_friend is the far more common conversation starter.
@I-am-Erk I-am-Erk changed the title NPC Dialogue Update 1.2: job-specific background "survivor stories" [WIP] NPC Dialogue Update 1.2: job-specific background "survivor stories" Dec 4, 2018
@ZhilkinSerg ZhilkinSerg added [JSON] Changes (can be) made in JSON NPC / Factions NPCs, AI, Speech, Factions, Ownership <Enhancement / Feature> New features, or enhancements on existing labels Dec 4, 2018
Scientist 1 was an analytical biochemist who got in over his head.
These tie into one of my downstream objectives, to make a player-run lab with NPCs staffing it. 

Next update I will probably divorce 'field of expertise' from the survival story, so each scientist is spawned with a different field - Biology, Chemistry, Physics - and associated dialogue and quest options for that. For now, I'm keeping it simple, one background trait per survivor.
@I-am-Erk I-am-Erk changed the title [WIP] NPC Dialogue Update 1.2: job-specific background "survivor stories" NPC Dialogue Update 1.2: job-specific background "survivor stories" Dec 13, 2018
@Rivet-the-Zombie Rivet-the-Zombie merged commit f06dea5 into CleverRaven:master Dec 15, 2018
@I-am-Erk I-am-Erk deleted the npc-1.4-specific-talk branch January 22, 2019 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Enhancement / Feature> New features, or enhancements on existing [JSON] Changes (can be) made in JSON NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants