-
-
Notifications
You must be signed in to change notification settings - Fork 699
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
Update burgs-and-states.js #1054
Conversation
Khanate title for Mongolian Empires and Kingdoms, Khaganate for Turkic.
✅ Deploy Preview for afmg ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I think there was a problem with Khanate and Khaganate's Form Name Generation. I've fixed it. |
I also added Beylik form name Generation.
modules/burgs-and-states.js
Outdated
@@ -859,9 +859,10 @@ window.BurgsAndStates = (function () { | |||
if (P(0.3) && s.diplomacy.includes("Vassal")) return "Protectorate"; // some vassals | |||
} | |||
|
|||
if (base === 16 && (form === "Empire" || form === "Kingdom")) return "Khaganate"; // Turkic | |||
if (base === 31 && (form === "Empire" || form === "Kingdom")) return "Khanate"; // Mongolian | |||
if (base === 31 && (form === "Principality" )) return "Beylik"; // Turkic |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
31 or 16 ?
Khanate and Khaganate is basically the same, but we can keep it separate it if want, just fix the id, please |
Fixed It! |
SOmebody Please Merge this. |
Actually need to update the version as well |
How do you do that? I'm kind of new. |
Not a problem, I can do it |
thank you. |
Khanate title for Mongolian Empires and Kingdoms, Khaganate for Turkic.
Description
Type of change
Versioning