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

Teleport other #3533

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Teleport other #3533

wants to merge 2 commits into from

Conversation

Loobinex
Copy link
Member

@Loobinex Loobinex commented Oct 7, 2024

No description provided.

@walt253
Copy link
Contributor

walt253 commented Oct 8, 2024

But it's already possible? You don't need to do all of this.

[shot112]
Name = SHOT_TIME_STAFF
SpellEffect = 10

In fact the above is all you need currently then your instance:

[instance140]
Name = TIME_STAFF
Time = 10
ActionTime = 8
ResetTime = 20
FPTime = 5
FPActionTime = 4
FPResetTime = 10
FPInstantCast = 0
ForceVisibility = 150
TooltipTextID = 673
SymbolSprites = 400
Graphics = ATTACK
RangeMin = MIN
RangeMax = 288
PrimaryTarget = 3
Properties = REPEAT_TRIGGER DESTRUCTIVE MELEE_ATTACK
Function = creature_fire_shot SHOT_TIME_STAFF 0

EDIT: but maybe there is an issue I'm not aware of with my method?

@walt253 walt253 marked this pull request as ready for review October 8, 2024 00:40
@walt253 walt253 marked this pull request as draft October 8, 2024 00:40
@walt253
Copy link
Contributor

walt253 commented Oct 8, 2024

For a potential teleport that targets allies, what about a rule so the users only target critical low health allies?

@Loobinex
Copy link
Member Author

But it's already possible? You don't need to do all of this.

[shot112]
Name = SHOT_TIME_STAFF
SpellEffect = 10

In fact the above is all you need currently then your instance:

[instance140]
Name = TIME_STAFF
Time = 10
ActionTime = 8
ResetTime = 20
FPTime = 5
FPActionTime = 4
FPResetTime = 10
FPInstantCast = 0
ForceVisibility = 150
TooltipTextID = 673
SymbolSprites = 400
Graphics = ATTACK
RangeMin = MIN
RangeMax = 288
PrimaryTarget = 3
Properties = REPEAT_TRIGGER DESTRUCTIVE MELEE_ATTACK
Function = creature_fire_shot SHOT_TIME_STAFF 0

EDIT: but maybe there is an issue I'm not aware of with my method?

Did you try it, did it work? Where would it teleport an enemy hero to?

@walt253
Copy link
Contributor

walt253 commented Oct 26, 2024

But it's already possible? You don't need to do all of this.

[shot112]
Name = SHOT_TIME_STAFF
SpellEffect = 10

In fact the above is all you need currently then your instance:

[instance140]
Name = TIME_STAFF
Time = 10
ActionTime = 8
ResetTime = 20
FPTime = 5
FPActionTime = 4
FPResetTime = 10
FPInstantCast = 0
ForceVisibility = 150
TooltipTextID = 673
SymbolSprites = 400
Graphics = ATTACK
RangeMin = MIN
RangeMax = 288
PrimaryTarget = 3
Properties = REPEAT_TRIGGER DESTRUCTIVE MELEE_ATTACK
Function = creature_fire_shot SHOT_TIME_STAFF 0

EDIT: but maybe there is an issue I'm not aware of with my method?

Did you try it, did it work? Where would it teleport an enemy hero to?

Yes it works. Enemy hero should teleport on their flee point/dungeon heart/hero gate.

BUT if there is none of that they will jump on PLAYER0 dungeon heart.

@Loobinex
Copy link
Member Author

Annoying, need to look into it a bit then.

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