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

DRAFT - Scourge random assign undead (second try) #1248

Closed
wants to merge 11 commits into from

Conversation

edmund-h
Copy link
Contributor

@edmund-h edmund-h commented Nov 22, 2023

DRAFT PULL REQUEST!

This is just a suggestion for possible changes, with peace and love ✌️ ➕ ❤️, dont take it seriously and please ping me with any suggestions on how to make it better

Changelog:

  • Lower acceptance for titles, marriage, vassalization, etc to LK (esp if in Union)
  • LK makes people undead by inviting them to court
  • LK makes people undead by granting titles
  • 90% chance for population in undead culture lands to be undead at game start.

Developer changelog:

change from previous pull request:

Tests:

  • There are no errors in wc files in Documents\Paradox Interactive\Crusader Kings III\logs\error.log except portrait_decals.cpp:101
  • The mod takes less than 5.5 GB in the Task Manager (Windows)

How to test:

@Grekopithikos
Copy link
Contributor

Isn't turning a character undead already an option you can have when giving a title to someone?
Why would the Lich King not be able to control whether they turn undead or not?

@edmund-h
Copy link
Contributor Author

edmund-h commented Nov 22, 2023

Isn't turning a character undead already an option you can have when giving a title to someone?

No it only currently happens when you vassalize someone, and it is in fact an option you have

Why would the Lich King not be able to control whether they turn undead or not?

you’re right, what i will do is make this optional in order to bring it in line with how vassalization works

@edmund-h edmund-h closed this Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants