-
Notifications
You must be signed in to change notification settings - Fork 228
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
Azazel Purchasables - Summons | 30€ Bounty #2111
Comments
I'd like to claim this one. |
Implementing this, progress can be tracked on PR #2156. Missing values used:
Would you guys prefer the items to show all their levels' values, or only the current level? e.g. for the level 2 tank summon, should it say "Tank Health: 10000/20000/30000/40000", or just "Tank Health: 20000"? |
All values. 300 MS seems fine, sorry for leaving that out. |
Finished this issue with PR #2156. Included some tooltips and placeholder icons. |
* Implementing #2110 * Added summon KV and extra abilities Assumed 300 MS for farmer and 1.0 turn rate for all summons. * Added azazel summon item entries and basic functionality * Finished implementing #2111 - Used "funny" names for the summons, just change them if you don't like them or think they're not clear enough. - Summon stats are tied to their actual stat block in npc_units_custom; the values in the item just change tooltips. - The Farmer's cleave stats are tied to the ability's KV, not the item's; again, the item's KVs just change tooltips. If the whole KV/tooltip thing is a bother, it can be changed, but I thought it was outside the scope of the issue, since the current behavior is easier to implement and consistent with (almost) every vanilla summon ability in the game. * travis * missing tooltip for modifier "dota_tooltip_modifier_azazel_summon_farmer_innate" - it's a hidden modifier that needs no tooltip. * item/ability ID conflicts - fixed * "cost is set correctly in kv for item_elixier_sustain_4" - I assume this means incorrectly. The items all cost 100g (recipe) + their core's cost (1500/2500/3500/4500) so I don't see the error here, but i set everything to 100 anyway, assuming cores have no real value. * "[item] has reverse compatible upgrade [core x]" - If I understood correctly, this is intended. * blah blah whitespace - fixed * blah unused variable - fixed Also made the items temporarily purchasable in the regular shop. * seems like travis' "next free ID" is a fat old lie fixed conflicting item/ability IDs * test test * Fix Travis
Azazel
Azazel is a fallen angel that runs a shop in the center area of the map. This is a major new feature. This issue contains the second set of purchasable items.
Unlike the regular shop, Azazel does not trade in gold. Instead all items in his shop are only available for purchase via cores. Purchasables listed as Tier 1 will require a Tier 1 core and a 100 gold recipe, Tier 2 requires a Tier 2 core and a 100g recipe, etc.
Summons
Summons are one time consumables that summon a controllable creature when used. These don’t get oneshot by charger, aren’t able to dominated or stolen away. They are considered heroes for all ability purposes. They can't be stacked in inventory and only one may be active at a time. Using a summon when another one is active destroys the previous one. We currently have 3 Summons and each Summon comes at 4 power levels. All Summons last 5 minutes or until killed.
The Summons
Tier 1
Tier 2
Tier 3
Tier 4
All summons need KVs with their corresponding stats.
Related Issues
Azazel Baseline functionality #2107
Azazel Model #2108
Azazel Voice #2109
Azazel Purchasables
Bug Bounty Guideline
The text was updated successfully, but these errors were encountered: