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

Update royal_city_of_rabanastre.js #18

Merged
merged 1 commit into from
Dec 4, 2017
Merged

Conversation

Vuzrak
Copy link

@Vuzrak Vuzrak commented Dec 3, 2017

Updated triggers for mask.

Updated triggers for mask.
@quisquous
Copy link
Owner

Ohmygosh, thanks! I had thought originally that the mask ability only occurred after the divine commandment debuff wore off, but double checking the logs that's not true at all.

As a side question, have you had any lag problems with triggers in Rabanastre with cactbot, especially on Hashmal or Argath?

@quisquous quisquous merged commit 0ca85da into quisquous:master Dec 4, 2017
@Vuzrak
Copy link
Author

Vuzrak commented Dec 5, 2017

No issues here with lag during Rabanastre, I do have TTS off so not sure if that would make a difference.

@Vuzrak Vuzrak deleted the patch-1 branch December 18, 2017 04:47
SiliconExarch pushed a commit to SiliconExarch/cactbot that referenced this pull request Jan 5, 2024
Closes quisquous#18.
(ref discussion in quisquous#5943).

Instead of ignoring all abilities used by combatants with an empty name
field, `make_timeilne` will now include entries in the timeline and the
timeline ability table if the empty-name combatant uses a named ability.
SiliconExarch pushed a commit to SiliconExarch/cactbot that referenced this pull request Jan 6, 2024
Closes quisquous#18.
(ref discussion in quisquous#5943).

Instead of ignoring all abilities used by combatants with an empty name
field, `make_timeilne` will now include entries in the timeline and the
timeline ability table if the empty-name combatant uses a named ability.
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.

3 participants