Hospitaller - IMPLEMENTED

Moderator: Endru1241

User avatar
makazuwr32
Posts: 4504
Joined: Tue Oct 17, 2017 9:29 am
Location: Moscow, Russia

Re: Hospitaller - ACCEPTED

Post by makazuwr32 » Sun Jul 08, 2018 4:55 am

It is unit by itself but in topic there is suggested the unit and its upgrade and research for it must be tied to knight templar.

User avatar
COOLguy
Posts: 4005
Joined: Sat Jul 12, 2014 2:58 am
Location: Nenuial, Arnor

Re: Hospitaller - ACCEPTED

Post by COOLguy » Sun Jul 08, 2018 2:40 pm

Ok that makes sense. Thanks
Thanks!
Josh

User avatar
StormSaint373
Posts: 1665
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 » Tue Aug 21, 2018 3:39 pm

This unit is currently by itself, but when using upgrades I think that the Defenders of the Faith, (Unit recruited from the church) should be in a set, like the Romans...

Would make things easier, in a way. But the Choice is entirely up to you.
Beware the calm before the Tempest. . .

User avatar
Gral.Sturnn
Posts: 1142
Joined: Tue Feb 13, 2018 11:29 pm

Re: Hospitaller - ACCEPTED

Post by Gral.Sturnn » Tue Aug 21, 2018 4:25 pm

What sprite shall be used?
~Gral.Sturnn

User avatar
makazuwr32
Posts: 4504
Joined: Tue Oct 17, 2017 9:29 am
Location: Moscow, Russia

Re: Hospitaller - ACCEPTED

Post by makazuwr32 » Tue Aug 21, 2018 5:01 pm

Both because one is normal one and second is upgrade.

User avatar
Gral.Sturnn
Posts: 1142
Joined: Tue Feb 13, 2018 11:29 pm

Re: Hospitaller - ACCEPTED

Post by Gral.Sturnn » Tue Aug 21, 2018 5:04 pm

Looks good
~Gral.Sturnn

User avatar
godOfKings
Posts: 2194
Joined: Fri Sep 22, 2017 4:50 pm

Re: Hospitaller - ACCEPTED

Post by godOfKings » Thu Sep 06, 2018 3:50 pm

How about the unit gets a separate upgrade tech instead of putting with knight templar?


Tech: To Knight Hospitaller
Cost: 4 turns (much lower upgrade cost for single unit upgrade)

Builder: church/cathedral
Attachments
tech_hospitaller_upgrade.png
tech_hospitaller_upgrade.png (272 Bytes) Viewed 937 times
There is no place for false kings here, only those who proves themselves to b the true kings of legend, or serves under me

For I watch over this world looking for those worthy to become kings, and on the way get rid of the fakes and rule over the fools

User avatar
StormSaint373
Posts: 1665
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 » Sat Sep 08, 2018 1:34 pm

That looks pretty accurate to the actual symbol of the order, OK you've got it!
Beware the calm before the Tempest. . .

User avatar
StormSaint373
Posts: 1665
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 » Mon Aug 26, 2019 6:51 pm

@Stratego (dev)

What else is needed for this?
Attachments
tech_upgrade_hospitaller_.json
(774 Bytes) Downloaded 7 times
unit_knight_hospitaller.json
(3.49 KiB) Downloaded 6 times
unit_hospitaller.json
(3.48 KiB) Downloaded 8 times
tech_hospitaller_upgrade.png
tech_hospitaller_upgrade.png (272 Bytes) Viewed 441 times
AoS Hospitaller, Knight.png
AoS Hospitaller, Knight.png (885 Bytes) Viewed 441 times
AoS Hospitaller.png
AoS Hospitaller.png (830 Bytes) Viewed 441 times
Beware the calm before the Tempest. . .

Stratego (dev)
Site Admin
Posts: 9680
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) » Wed Aug 28, 2019 5:19 am

is it a standalone unit "hospitaller"
and there is a tech "tech_upgrade_hospitaller" that can upgrade it to "hospitaller knight"

right?

(jsons have some wrong setting tell me if i need to list them here or if i only need to fix them and put in :) )

User avatar
StormSaint373
Posts: 1665
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 » Wed Aug 28, 2019 2:39 pm

Correct.
Beware the calm before the Tempest. . .

User avatar
StormSaint373
Posts: 1665
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 » Wed Aug 28, 2019 9:19 pm

Unfortunately, @Stratego (dev) , I only know how to edit numbers...

What else needs to be done?
Beware the calm before the Tempest. . .

Stratego (dev)
Site Admin
Posts: 9680
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) » Thu Aug 29, 2019 4:43 am

i will fix the json and you will see if you compare it to yours - ok?

User avatar
StormSaint373
Posts: 1665
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 » Fri Aug 30, 2019 12:40 pm

Ok
Beware the calm before the Tempest. . .

Stratego (dev)
Site Admin
Posts: 9680
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) » Thu Sep 05, 2019 8:55 am

i still had no time as i am doing online multiplayer logic coding, if anyone have time please fix the jsons
(as i see they are now mostly archer and crossbow jsons barely modified)

best if someone can try with modding before sending to me - thanks!

User avatar
StormSaint373
Posts: 1665
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 » Thu Sep 05, 2019 10:18 am

Performed minor alterations to match Templar
Attachments
AoS Hospitaller, Knight.png
AoS Hospitaller, Knight.png (936 Bytes) Viewed 373 times
AoS Hospitaller.png
AoS Hospitaller.png (861 Bytes) Viewed 373 times
Beware the calm before the Tempest. . .

User avatar
StormSaint373
Posts: 1665
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 » Thu Sep 05, 2019 10:20 am

@Endru1241 , perhaps you can aid in this Json...

I still haven't grasped jsons appropriately yet.
Beware the calm before the Tempest. . .

User avatar
Endru1241
Posts: 496
Joined: Fri Sep 11, 2015 8:43 am
Location: Poland

Re: Hospitaller - ACCEPTED

Post by Endru1241 » Thu Sep 05, 2019 10:50 am

I corrected it. Now gonna test if works properly.
He had too much heal - changed it to 4 ( and at range 1) and 8 for upgraded one.
Please change names of graphics to unit_hospitaller.png and unit_hospitaller_upg.png


Edit - actually posted wrong file, also names of graphic assets
Last edited by Endru1241 on Thu Sep 05, 2019 2:31 pm, edited 4 times in total.

Stratego (dev)
Site Admin
Posts: 9680
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) » Thu Sep 05, 2019 12:31 pm

AWESOME you are!
remark: the "32_" should not be in image names - only very old assets have that (i ran into problems when i changed unit image to be 64px and i had to rename everywhere, so please dont put image size information in image names - thanks!)

User avatar
StormSaint373
Posts: 1665
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 » Thu Sep 05, 2019 12:36 pm

Shouldn't the heal Rate be the same as Templars?
Beware the calm before the Tempest. . .

User avatar
Endru1241
Posts: 496
Joined: Fri Sep 11, 2015 8:43 am
Location: Poland

Re: Hospitaller - ACCEPTED

Post by Endru1241 » Thu Sep 05, 2019 6:59 pm

Not sure - I wanted to test first if lower heal will be enough, but I felt 5 on base, and especially 10 on upgrade is too much (he would take no damage after heal from single basic skirmisher, so lowered a little. And upgraded hospitaller with 8 heal is set to mitigate master skirmisher damage instead of slowly healing).

Stratego (dev)
Site Admin
Posts: 9680
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) » Thu Sep 05, 2019 7:45 pm

awesome! can usend me jsons to implement?

User avatar
Endru1241
Posts: 496
Joined: Fri Sep 11, 2015 8:43 am
Location: Poland

Re: Hospitaller - ACCEPTED

Post by Endru1241 » Thu Sep 05, 2019 7:56 pm

Jsons + correctly named pngs.
Attachments
tech_hospitaller_upgrade.png
tech_hospitaller_upgrade.png (272 Bytes) Viewed 346 times
unit_hospitaller_upg.png
unit_hospitaller_upg.png (936 Bytes) Viewed 346 times
unit_hospitaller.png
unit_hospitaller.png (861 Bytes) Viewed 346 times
tech_upgrade_hospitaller.json
(796 Bytes) Downloaded 1 time
unit_hospitaller_upg.json
(3.49 KiB) Downloaded 2 times
unit_hospitaller.json
(3.43 KiB) Downloaded 2 times

Stratego (dev)
Site Admin
Posts: 9680
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) » Thu Sep 05, 2019 8:00 pm

but as i understood you had no chance the test, i upload new dev version in 5 minutes so u can test prior.

User avatar
Endru1241
Posts: 496
Joined: Fri Sep 11, 2015 8:43 am
Location: Poland

Re: Hospitaller - ACCEPTED

Post by Endru1241 » Thu Sep 05, 2019 8:57 pm

Stratego (dev) wrote:
Thu Sep 05, 2019 8:00 pm
but as i understood you had no chance the test, i upload new dev version in 5 minutes so u can test prior.
Now tested - it works - he shoots, he kills, he heals and he upgrades properly - only in unit details icons are still missing (and texts, but that I understand why).

Stratego (dev)
Site Admin
Posts: 9680
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) » Fri Sep 06, 2019 5:02 am

awesome you are!

User avatar
Endru1241
Posts: 496
Joined: Fri Sep 11, 2015 8:43 am
Location: Poland

Re: Hospitaller - ACCEPTED

Post by Endru1241 » Fri Sep 06, 2019 10:12 pm

Actually, after using him few times I noticed that he could be a little more resilient and more importantly it's a little too weak to matter to other units -I would change that heal +2(6) /+1(9).
Also shadow look weird.
@StormSaint373 Are you willing to correct the shadow?
Attachments
unit_hospitaller.json
(3.46 KiB) Downloaded 3 times
unit_hospitaller_upg.json
(3.53 KiB) Downloaded 2 times

User avatar
Puss_in_Boots
Posts: 2732
Joined: Fri Apr 21, 2017 1:23 am

Re: Hospitaller - ACCEPTED

Post by Puss_in_Boots » Fri Sep 06, 2019 11:01 pm

My edits.
Attachments
unit_hospitaller_upg.png
unit_hospitaller_upg.png (915 Bytes) Viewed 315 times
unit_hospitaller.png
unit_hospitaller.png (890 Bytes) Viewed 315 times
RED, WHITE and BLUE!

User avatar
StormSaint373
Posts: 1665
Joined: Mon Nov 27, 2017 5:35 pm
Location: USA, East Ohio

Re: Hospitaller - ACCEPTED

Post by StormSaint373 » Sat Sep 07, 2019 3:00 am

Ah, I see... Thx, @Puss_in_Boots .
Beware the calm before the Tempest. . .

Stratego (dev)
Site Admin
Posts: 9680
Joined: Fri Apr 25, 2014 9:28 pm

Re: Hospitaller - ACCEPTED

Post by Stratego (dev) » Sat Sep 07, 2019 11:35 am

Thanks!

all is in!

mega thanks!

Post Reply

Return to “Unit - ranged (like archers)”