Differential D1806 Diff 9368 binaries/data/mods/public/simulation/data/auras/structures/wonder_pop_2.json
Changeset View
Changeset View
Standalone View
Standalone View
binaries/data/mods/public/simulation/data/auras/structures/wonder_pop_2.json
{ | { | ||||
"type": "player", | "type": "player", | ||||
"affects": ["Player"], | "affects": ["Player"], | ||||
"modifications": [ | "modifications": [ | ||||
{ "value": "Player/MaxPopulation", "add": 40 } | { "value": "Player/MaxPopulation", "add": 40 } | ||||
], | ], | ||||
"auraName": "Glorious Expansion", | "auraName": "Glorious Expansion", | ||||
"auraDescription": "Further increase the population limit by 40 per wonder owned (requires \"Glorious Expansion\" tech).", | "auraDescription": "40 maximum population limit (requires the “Glorious Expansion” technology).", | ||||
Gallaecio: 40.0 → +40
Shouldn’t “per wonder owned” remain in the description? | |||||
Done Inline ActionsOops, missed that .0, will correct. And no, we don't have that in other stackable auras (e.g. library, theatre) either. Nescio: Oops, missed that `.0`, will correct.
And no, we don't have that in other stackable auras (e.g. | |||||
Not Done Inline ActionsI think the + in +40 got accidentally lost in the last change. Gallaecio: I think the + in +40 got accidentally lost in the last change. | |||||
Not Done Inline ActionsSince we are changing the string already, maybe we could switch to ‘requires the “Glorious Expansion” technology’ (‘the’ added after ‘requires’). Just ‘requires “Glorious Expansion”’ would also work, but I think making clear that it is a ‘technology’ is very important here. Gallaecio: Since we are changing the string already, maybe we could switch to ‘requires the “Glorious… | |||||
"requiredTechnology": "pop_wonder", | "requiredTechnology": "pop_wonder", | ||||
"stackable": true | "stackable": true | ||||
} | } |
Wildfire Games · Phabricator
40.0 → +40
Shouldn’t “per wonder owned” remain in the description?