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

Clarify durations in NPCs.md #27704

Merged
merged 2 commits into from
Jan 18, 2019

Conversation

I-am-Erk
Copy link
Member

Also adds examples.

Summary

SUMMARY: Bugfixes "Fixes documentation of 'duration' feature in NPCs.md"

Purpose of change

The issue caused in #27699 is related to a mistake I made on a lot of my earlier NPCs, where I added durations as described in the documentation, which was incorrect.

Describe the solution

Clarified in documentation that duration is not optional, and that the only permitted string is "PERMANENT", other durations passed are integers.

Also added examples, because examples are good.

Also adds examples.
@I-am-Erk I-am-Erk changed the title Clarify durations in NPC.md Clarify durations in NPCs.md Jan 18, 2019
@mlangsdorf
Copy link
Contributor

Since #27706 is a better solution to this, can you delete the doc text change but keep the new examples?

@mlangsdorf mlangsdorf added <Documentation> Design documents, internal info, guides and help. <Bugfix> This is a fix for a bug (or closes open issue) NPC / Factions NPCs, AI, Speech, Factions, Ownership labels Jan 18, 2019
@I-am-Erk
Copy link
Member Author

I-am-Erk commented Jan 18, 2019

Yep, as soon as I'm at a desktop computer

fix to allow mlangsdorf's changes
@ZhilkinSerg ZhilkinSerg merged commit 6ba43b1 into CleverRaven:master Jan 18, 2019
@I-am-Erk I-am-Erk deleted the npcmd-duration-fix branch January 22, 2019 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) <Documentation> Design documents, internal info, guides and help. NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants