Adds Map of China#4468
Conversation
This reverts commit be91d86.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
WalkthroughAdds a new China map across generator data, runtime manifest data, registry wiring, translation, and country flag entries. It also updates CaspianSea categories and reformats the CaspianSea manifest. ChangesChina Map Addition
Estimated code review effort: 2 (Simple) | ~15 minutes Possibly related PRs
Suggested labels: Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@map-generator/assets/maps/china/info.json`:
- Around line 270-274: Update the map entry for Second East Turkestan Republic
to use the existing Eastern_Turkistan_Flag asset instead of the pk country code.
Make this change in the China map info data and keep the corresponding manifest
entry in sync so both references point to the same flag identifier. Locate the
affected records by the name field "Second East Turkestan Republic" and the
Eastern_Turkistan_Flag symbol already defined in the repo.
- Around line 275-279: The Shanxi Clique entry in the map info uses the wrong
flag symbol. Update the record identified by the name field “Shanxi Clique” so
it uses the Republic of China flag symbol instead of “tw”, matching the other
cliques in this dataset.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: f31e2549-0585-40d4-b866-5b7d3907dae4
⛔ Files ignored due to path filters (10)
map-generator/assets/maps/china/image.pngis excluded by!**/*.pngresources/flags/Eastern_Turkistan_Flag.svgis excluded by!**/*.svgresources/flags/Flag_of_Hong_Kong_(1959-1997).svgis excluded by!**/*.svgresources/flags/Flag_of_Manchukuo.svgis excluded by!**/*.svgresources/flags/Flag_of_Taiping_Heavenly_Kingdom.svgis excluded by!**/*.svgresources/flags/Kuomintang_Emblem.svgis excluded by!**/*.svgresources/flags/Ma_Clique_Flag.svgis excluded by!**/*.svgresources/maps/china/map.binis excluded by!**/*.binresources/maps/china/map16x.binis excluded by!**/*.binresources/maps/china/map4x.binis excluded by!**/*.bin
📒 Files selected for processing (8)
map-generator/assets/maps/caspiansea/info.jsonmap-generator/assets/maps/china/info.jsonresources/countries.jsonresources/lang/en.jsonresources/maps/caspiansea/manifest.jsonresources/maps/china/manifest.jsonresources/maps/china/thumbnail.webpsrc/core/game/Maps.gen.ts
Add approved & assigned issue number here:
Resolves #4419
Description:
https://youtu.be/ew9Qizo67cA
Please complete the following:
Please put your Discord username so you can be contacted if a bug or regression is found:
DISCORD_USERNAME crunchybbbbb