← Back to team overview

widelands-dev team mailing list archive

[Merge] lp:~widelands-dev/widelands-website/map_version_gt_19 into lp:widelands-website

 

kaputtnik has proposed merging lp:~widelands-dev/widelands-website/map_version_gt_19 into lp:widelands-website.

Requested reviews:
  Widelands Developers (widelands-dev)
Related bugs:
  Bug #1731732 in Widelands Website: "Check whether a map needs a version of Widelands newer than Build 18"
  https://bugs.launchpad.net/widelands-website/+bug/1731732

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands-website/map_version_gt_19/+merge/333591

Add info for maps that requires a build > build19, depending on the number of players.
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands-website/map_version_gt_19 into lp:widelands-website.
=== modified file 'templates/wlmaps/index.html'
--- templates/wlmaps/index.html	2017-08-16 21:13:43 +0000
+++ templates/wlmaps/index.html	2017-11-12 16:18:54 +0000
@@ -56,7 +56,13 @@
 					</tr>
 					{% if not map.world_name %}
 					<tr>
-						<td colspan="5"><strong>This map requires a version of Widelands newer than build18!</strong></td>
+						<td colspan="5"><strong>This map requires a version of Widelands newer than build
+						{% if map.nr_players > 8 %}
+							19!
+						{% else %}
+							18!
+						{% endif %}
+						</strong></td>
 					</tr>
 					{% endif %}
 					<tr>

=== modified file 'templates/wlmaps/map_detail.html'
--- templates/wlmaps/map_detail.html	2017-04-25 19:49:02 +0000
+++ templates/wlmaps/map_detail.html	2017-11-12 16:18:54 +0000
@@ -108,7 +108,13 @@
 		
 		{% if not map.world_name %}
 		<div>
-			<strong>This map requires a version of Widelands newer than build18!</strong>
+			<strong>This map requires a version of Widelands newer than build
+            {% if map.nr_players > 8 %}
+                19!
+            {% else %}
+                18!
+            {% endif %}
+            </strong>
 		</div>
 		{% endif %}
 	</div>


Follow ups