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

Fix order template #210

Merged
merged 2 commits into from
Sep 18, 2024
Merged

Fix order template #210

merged 2 commits into from
Sep 18, 2024

Conversation

jt-traub
Copy link
Contributor

Fix bug where a passed-through hex showed up in order template

@@ -4978,7 +4978,7 @@ static TerrainType td[] = {
{-1,-1,-1},
0,-1,-1,-1,
0,{-1,-1,-1,-1,-1,-1}},
{"ocean", "ocean", "barrenocean", '-', R_OCEAN,
{"deadwater", "deadwater", "barrenocean", '-', R_OCEAN,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change is actually already in master.. it will merge cleanly. Github is just confused because I froze the NO v7 stable branch and am merging from a branch off of that onto master. The only real change in this commit is the change on line 900 in text_report_generator.cpp

@jt-traub jt-traub merged commit 7bee78a into master Sep 18, 2024
15 checks passed
@jt-traub jt-traub deleted the jt-fix-order-template branch September 18, 2024 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants