Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.
Sign upMake mission_id 78 solvable #17077
Conversation
mlangsdorf
closed this
Jun 6, 2016
mlangsdorf
reopened this
Jun 6, 2016
This comment has been minimized.
This comment has been minimized.
|
This address the bug reported here |
This comment has been minimized.
This comment has been minimized.
|
The best way to handle issue fixing is to edit the original post to put in the exact phrase: Which will ensure that if your pull request gets merged that it automatically closes that issue. Elsewise we have to find that issue and close it manually. |
This comment has been minimized.
This comment has been minimized.
|
On Tue, Jun 7, 2016 at 2:50 PM, Christian Buskirk notifications@github.com
Does the "original post" mean the thread here: or my pull request here: or something else? And how do I get the issue number?
I want to contribute as usefully as possible, I'm just not at all familiar Thanks for the pointers. --Mark Lagnsdorf |
This comment has been minimized.
This comment has been minimized.
|
#17077 is this very PR we're discussing, I think you made a typo. |
This comment has been minimized.
This comment has been minimized.
|
I will edit the PR title to add the bug issue number but I don't know the bug issue number. How do I get that information? |
Coolthulhu
self-assigned this
Jun 8, 2016
This comment has been minimized.
This comment has been minimized.
|
I can't test it, NPCs with this mission don't seem to want to give it. Mission code is a major mess so there's a chance missions are sorted by ID, need extra changes or something like that. |
Coolthulhu
removed their assignment
Jun 8, 2016
This comment has been minimized.
This comment has been minimized.
|
I branched my tree off of commit f9fa31f and at that point it was possible to test the mission using the debug key. I'm not sure what happened since then. |
This comment has been minimized.
This comment has been minimized.
|
Well, the NPC gives out the mission once they follow you. And the mission itself works fine. |
BevapDin
reviewed
Jun 10, 2016
| @@ -2496,6 +2496,32 @@ Before we get into a major fight just make sure we have the gear we need, boss." | |||
| } | |||
| break; | |||
|
|
|||
| case MISSION_FIND_ZSG: | |||
| if( state == "TALK_MISSION_DESCRIBE" ) { | |||
| return _("I've can an idea about how to improve our chances of surviving"); | |||
This comment has been minimized.
This comment has been minimized.
BevapDin
Jun 10, 2016
Contributor
"can an idea"? Maybe "I've an idea..." - the "can" seems wrong there. There seems to be a period missing at the end of the sentence.
This comment has been minimized.
This comment has been minimized.
mlangsdorf
Jun 10, 2016
Author
Contributor
Thanks for the feedback.
I've made the changes in my local tree to fix those two typos - how do I go about publishing them? Do I amend the commit and push the fix back to github or something else? I checked
http://smf.cataclysmdda.com/index.php?topic=1063.0 but it didn't address the issue of making changes to your code after submitting the pull request.
This comment has been minimized.
This comment has been minimized.
Zireael07
Jun 11, 2016
Contributor
You push to your branch (mlangsdorf:fix_mission_78) and this PR automatically gets updated.
Pro tip: use a GUI client for git stuff, such as SourceTree or GitHub for Windows.
mlangsdorf commentedJun 6, 2016
Change mission_id 78 from "Find a Book" to "Find the Zombie Guide",
add quest dialog, and make it solvable with a copy of the Zombie
Survival Guide.