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

Various Corrections of Spelling and Clarification of Quests #31216

Merged
merged 5 commits into from
Jun 12, 2019

Conversation

DracoGriffin
Copy link
Contributor

Summary

SUMMARY: None

Purpose of change

Correct typos and update spelling to US English per manual. Altered quests to be less redundant and more clear in the objective.

Additional context

Quest related --- quests have for NPC_quest_giver appended to the end of quest names automatically in-game.

For Fatima's quest:
"name": "Find a copy of the Quran for Fatima",
shows up as
Find a copy of the Quran for Fatima for Fatima al Jadir, refugee
quranfatimafatima
After fix will show up as:
Find a copy of the Quran for Fatima al Jadir, refugee

Likewise for fetch missions, having unclear items and amounts is annoying, mainly for Dino Dave:
"name": "Bring Dino Dave small cardboard boxes.",
howmanyboxes
After fix will show up as:
Bring 40 small cardboard boxes for Dino Dave, beggar

Some of these could be re-worded to be slightly more clear, for instance:
Bring 40 small cardboard boxes for Dino Dave, beggar
could be changed to
Bring back 40 small cardboard boxes for Dino Dave, beggar

but wasn't sure how needful that may be.

- Correct typos
- Reformat quest names to be clearly identify what is required and remove questgiver (quests already specify the questgiver - "Bring items to John Doe for John Doe")
- Replace UK English with US English as specified by MANUAL_OF_STYLE.md
- Format quests so they don't read as "Do X for NPC Y for NPC Y", and now properly read as "Do X for NPC Y"
- Update UK English with US English
- Clarified quest items in quest for actual item needed rather than being unclear
@DracoGriffin DracoGriffin added NPC / Factions NPCs, AI, Speech, Factions, Ownership <Bugfix> This is a fix for a bug (or closes open issue) labels Jun 7, 2019
@ZhilkinSerg ZhilkinSerg self-assigned this Jun 10, 2019
@kevingranade kevingranade merged commit dca7f08 into CleverRaven:master Jun 12, 2019
@ZhilkinSerg ZhilkinSerg removed their assignment Jun 13, 2019
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) NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants