Skip to content
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

Gundoaldus, Lombard Dux of Asti c.589-612 #376

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

IhateTrains
Copy link
Collaborator

I'm making a PR because many files are edited and they should be reviewed by someone else.

@IhateTrains IhateTrains requested a review from loup99 April 18, 2018 16:46
@IhateTrains IhateTrains added Flavour Additions to enhance the mod and make it more immersive. History Everything in relation to history. labels Apr 18, 2018
@@ -0,0 +1,3 @@
589.1.1={
holder = 190180 #Gundualdus
}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the barony?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wanted to give them c_monferrato, but the county is already owned by another Lombard duchy (d_monferrato). d_asti holding b_asti seems the best solution in terms of representing both duchies at once.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

b_asti is a city, as per the historical situation.

Copy link
Collaborator Author

@IhateTrains IhateTrains Apr 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see two solutions:

  • let Asti have only b_asti and be a republic
  • activate one of the inactive castle-type baronies of c_monferrato and give it to Asti

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the first:

  • A land-locked republic? Remember that is unplayable, so completely useless for the player.

Copy link
Collaborator Author

@IhateTrains IhateTrains Apr 20, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added this character mostly for historical accuracy. Asti being a county-less republic wouldn't block the player from playing as "feudal" Monferrato.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, but I don't get you, could you please explain further? How would it be historically accurate to create a land-locked republic lead by a historical feudal dynasty?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's historically accurate in terms of the duchy of Asti existing. For "feudal" government, I can give it one castle-type barony.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then the issue is that we would make the county more powerful. I don't know if that is adequate during this period.

Copy link
Collaborator Author

@IhateTrains IhateTrains Apr 21, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know either.
What I have found is that Tortona (b_tortona) remained quite significant after the fall of the WRE. Even Theodoric the Great called it "the granary of Liguria". I guess we could activate it.

@IhateTrains
Copy link
Collaborator Author

I put the PR on hold until we merge the Holy Fury (and previous) vanilla map changes.

@loup99 loup99 added this to the 0.9.13 milestone Jan 6, 2019
@loup99 loup99 removed this from the 0.9.13 milestone Apr 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Flavour Additions to enhance the mod and make it more immersive. History Everything in relation to history.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants