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

[Core] Add Rez Macro #405

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

[Core] Add Rez Macro #405

wants to merge 11 commits into from

Conversation

zbee
Copy link
Member

@zbee zbee commented Mar 16, 2025

Important

This should spend considerable time in testing before even upgrading it to party chat, let alone merging.
Please do not approve unless you have actually used this in-game.

Note

Currently this will ONLY output to echo chat no matter what, for testing.

This PR adds a setting to setup a Rez Macro that will then be output any time a Rez is cast by Wrath.
The macro is customizable, and supports <placeholders>.
The macro cannot run more than once every 3 seconds, or 5 seconds for the same target, and has a lot more logic to prevent it from being ran (like requiring combat [for non-debug builds], and the rez to have just been used).

  • Add Setting for enabling rez macros
    image
    Helpmarker:
    image
  • Add Setting for modifying the rez macro
    image
    Helpmarker:
    image
  • Rez macro logic gauntlet to prevent spam and only print the macro when a rez is happening
  • Rez macro string forming and message sending
  • Hook Rez macro up to rezzes
    • Test button next to setting
    • All Healer Rez feature
    • All Caster Rez feature
    • WHM
    • AST
    • SCH
    • SGE
    • RDM
    • SMN
  • Output exactly as a macro would
    (Currently only outputs the player name, not even linked, due to a limitation in ECommons.Automation.Chat.Instance.SendMessage. I will modify the method, or create another, to allow sending the proper player link payloads.)

Closes #283

Tests

Date Class(es)
2025-03-16 All.Healer, WHM

@zbee zbee added the New Feature This adds a new option or feature label Mar 16, 2025
@PunishXIV PunishXIV deleted a comment from semanticdiff-com bot Mar 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Feature This adds a new option or feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Healers] Print-to-chat 'rez macro'?
1 participant