Conversation
Changes by create-pull-request action
|
Leader voicelines needed |
jsons/Units.json
Outdated
| "uniques": [ | ||
| "[+2] Strength <for every [[Happiness] / [5]]> <hidden from users>", | ||
| "Comment [+2 Strength for every 5 [Happiness]]", | ||
| "Comment [Unlocked earlier at [Dynamite] vs [Railroads]]", |
There was a problem hiding this comment.
Remove this comment (as the UI should automatically generate it, and if it doesn't, that's an Unciv problem
There was a problem hiding this comment.
Here I meant the comment about which tech unlocked it, not the comment about the unit strength
jsons/Nations.json
Outdated
| "outerColor": [174, 57, 36], | ||
| "innerColor": [141, 176, 75], |
There was a problem hiding this comment.
Time for me to get picky about colors (unless it told you to change them already)
Outer: 51, 28, 63
Inner: 236, 226, 79
There was a problem hiding this comment.
Wait, no. That's columbia's colors. Hold up, let me get the right ones
There was a problem hiding this comment.
Outer: 188, 27, 10
Inner: 123, 207, 0
| "spyNames": [ | ||
| "","","","","","","","","","" | ||
| ] |
There was a problem hiding this comment.
| "spyNames": [ | |
| "","","","","","","","","","" | |
| ] | |
| "spyNames": [ | |
| "Olaneta","Ayala","Ballivian","Paredes","Lanza","Perez","Azurduy","Pacheco","Morales" | |
| ] |
There was a problem hiding this comment.
Isn't there one spy name missing?
There is only 9 here.
There was a problem hiding this comment.
Missed Pinto, between Ballivian and Paredas
|
Lastly, please delete the atlases you've made. They should generate themselves automatically once this is merged |
|
Remove the comment for the required tech for the UU and this should be good to merge (I believe you got confused, that was the comment I was saying was unnecessary, though both wasn't needed) |
2 participants
Adds Bolivia, from Lekmod