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

[Quest API] Add TrackNPC to Perl/Lua. #2272

Merged
merged 3 commits into from
Jun 29, 2022
Merged

Conversation

Kinglykrab
Copy link
Contributor

  • Add quest::tracknpc(entity_id) to Perl.
  • Add eq.track_npc(entity_id) to Lua.
  • This will allow server operators to arbitrarily Track NPCs for clients with scripts.
  • Modified tracking to auto turn off and tell you that you discovered your target if within 10 units, this is to allow scripted Tracking to turn off without the need for the Tracking skill.
  • @fryguy503 mentioned a Live quest that uses this methodology to track an NPC on a Warrior, a class that doesn't have Tracking.

@Kinglykrab Kinglykrab force-pushed the quest_api/track_npc branch 3 times, most recently from 2e94215 to 571c152 Compare June 19, 2022 02:33
- Add quest::tracknpc(entity_id) to Perl.
- Add eq.track_npc(entity_id) to Lua.
- This will allow server operators to arbitrarily Track NPCs for clients with scripts.
- Modified tracking to auto turn off and tell you that you discovered your target if within 10 units, this is to allow scripted Tracking to turn off without the need for the Tracking skill.
Copy link
Contributor

@fryguy503 fryguy503 left a comment

Choose a reason for hiding this comment

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

LGTM Testing worked well

@Kinglykrab Kinglykrab merged commit 15113f4 into master Jun 29, 2022
@Kinglykrab Kinglykrab deleted the quest_api/track_npc branch June 29, 2022 19:29
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