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

Greymane bloodline #1701

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

Greymane bloodline #1701

wants to merge 17 commits into from

Conversation

aybarsavci
Copy link
Contributor

@aybarsavci aybarsavci commented Jul 15, 2020

Changelog:

  • Added Greymane Bloodline. Founder Is Archibald Greymane, father of Genn.

How to test:

Right Click on Gilneas, you will see greymane have a bloodline.

aybarsavci and others added 9 commits July 15, 2020 01:10
Added Greymane Bloodline (see from a suggestion)

Go Down to see it if you cant find.

have a nice day :)
Create 00_greymane bc localisation just get corrupted then i just create a new a little .csv file with only new things.

DO NOT COPY ZUCW FILE ITS ALL CORRUPTED, LET THIS FILE STAY OR COPY THINGS IN THIS FILE TO ZUCW FILE (bc i just cant its getting corrupted)
@aybarsavci aybarsavci requested a review from a team as a code owner July 15, 2020 15:01
@arithon arithon added the ⭐ new feature 🆕 Adding a new feature to the mod label Jul 15, 2020
@zumbak04
Copy link
Member

Fixed the changelog.

@@ -0,0 +1 @@
15.7.2020 Added Greymane bloodline
Copy link
Member

Choose a reason for hiding this comment

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

What is it?

Copy link
Member

Choose a reason for hiding this comment

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

Bang! Not addressed.

Comment on lines +1 to +2
greymane;Blood of [This.GetFirstNameWithNick];Sang de [This.GetFirstNameWithNick];Blut von [This.GetFirstNameWithNick];;;;;;;;;;;x
greymane_desc;Archibald Greymane was the king of Gilneas and the father of Genn Greymane. Under his rule, Gilneas prospered. His views on asking for help were that it was for the weak.;;;;;;;;;;;;;x
Copy link
Member

Choose a reason for hiding this comment

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

What is a point of this file? These lines exist in localisation/00_zuwc_united_localisation.csv.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i accidently uploaded the test file.

Copy link
Member

Choose a reason for hiding this comment

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

Bang! The file still exists.

zumbak04 and others added 5 commits July 15, 2020 19:24
(GDC)
This reverts commit c0da000.
it was a test file i accidently commit.
@aybarsavci aybarsavci requested a review from zumbak04 July 16, 2020 14:36
history/characters/46000_gilnean.txt Outdated Show resolved Hide resolved
common/bloodlines/wc_bloodlines.txt Outdated Show resolved Hide resolved
@aybarsavci aybarsavci requested a review from zumbak04 July 22, 2020 23:03
@zumbak04 zumbak04 added the ❕ priority medium Issue or addition is medium priority label Jul 23, 2020
@zumbak04 zumbak04 added this to the Patch 1.9.0 milestone Jul 23, 2020
…oodline

# Conflicts:
#	localisation/00_zuwc_united_localisation.csv
gilnean_opinion = 10

local_revolt_risk = 0.05
Copy link
Member

Choose a reason for hiding this comment

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

local_revolt_risk doesn't work with characters, no?
https://ck2.paradoxwikis.com/Modifiers

local_revolt_risk = 0.05

local_tax_modifier = 0.05
Copy link
Member

Choose a reason for hiding this comment

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

local_tax_modifier doesn't work with characters, no?
https://ck2.paradoxwikis.com/Modifiers

monthly_character_prestige = 0.15

gilnean_opinion = 10
Copy link
Member

Choose a reason for hiding this comment

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

The opinion is not declared in Warcraft-Guardians-of-Azeroth\common\modifier_definitions\wc_opinion_modifier_definitions.txt
See how other opinions are handled.

@zumbak04 zumbak04 removed this from the Patch 1.9.0 milestone Aug 28, 2020
@zumbak04 zumbak04 added this to the Soon™ milestone Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
❕ priority medium Issue or addition is medium priority ⭐ new feature 🆕 Adding a new feature to the mod
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants