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] (Performance) Check event EVENT_LANGUAGE_SKILL_UP, EVENT_SKILL_UP, or EVENT_USE_SKILL exist before export and execute #2894

Merged
merged 2 commits into from Feb 13, 2023

Conversation

Kinglykrab
Copy link
Contributor

@Kinglykrab Kinglykrab commented Feb 12, 2023

Notes

  • Optionally parse these events instead of always doing so.

@Kinglykrab Kinglykrab changed the title [Quest API] Optionally parse EVENT_LANGUAGE_SKILL_UP, EVENT_SKILL_UP,… [Quest API] Optionally parse EVENT_LANGUAGE_SKILL_UP, EVENT_SKILL_UP, and EVENT_USE_SKILL Feb 12, 2023
@Kinglykrab Kinglykrab changed the title [Quest API] Optionally parse EVENT_LANGUAGE_SKILL_UP, EVENT_SKILL_UP, and EVENT_USE_SKILL [Quest API] (Performance) Check event EVENT_LANGUAGE_SKILL_UP, EVENT_SKILL_UP, or EVENT_USE_SKILL exist before export and execute Feb 13, 2023
… and EVENT_USE_SKILL

- Optionally parse these events instead of always doing so.
@Akkadius Akkadius force-pushed the quest_api/skill_events_optional branch from 2ab1e81 to 4044f68 Compare February 13, 2023 05:30
@Akkadius Akkadius merged commit 3474c00 into master Feb 13, 2023
@Akkadius Akkadius deleted the quest_api/skill_events_optional branch February 13, 2023 05:30
@Akkadius Akkadius mentioned this pull request Feb 16, 2023
catapultam-habeo pushed a commit to catapultam-habeo/pyrelight-server that referenced this pull request Mar 27, 2023
…SKILL_UP, or EVENT_USE_SKILL exist before export and execute (EQEmu#2894)

* [Quest API] Optionally parse EVENT_LANGUAGE_SKILL_UP, EVENT_SKILL_UP, and EVENT_USE_SKILL

- Optionally parse these events instead of always doing so.

* Cleanup
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.

None yet

2 participants