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

raidboss: FRU p4 triggers #601

Merged
merged 8 commits into from
Feb 22, 2025
Merged

raidboss: FRU p4 triggers #601

merged 8 commits into from
Feb 22, 2025

Conversation

wexxlee
Copy link
Collaborator

@wexxlee wexxlee commented Feb 16, 2025

~~Marking this as WIP for now because I want to get it posted for feedback but may not be able to get back to it until next week. ~~

Working versions of these triggers have been tested quite a bit in game, but I changed a few things before PRing so I'm planning to re-test in game next week. (Probably also some typos in the comments, but I need to look again with fresh eyes.)

@github-actions github-actions bot added the raidboss /ui/raidboss module label Feb 16, 2025
Copy link
Collaborator

@JLGarber JLGarber left a comment

Choose a reason for hiding this comment

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

That all looks great on a first-pass read.

@wexxlee wexxlee marked this pull request as ready for review February 20, 2025 05:31
@github-actions github-actions bot added the needs-review Awaiting review label Feb 20, 2025
@wexxlee
Copy link
Collaborator Author

wexxlee commented Feb 20, 2025

Made some very minor delay/duration adjustments after re-testing in game and checking vod, but otherwise this is working as expected and should be ready.

@wexxlee wexxlee requested a review from JLGarber February 20, 2025 05:33
Copy link
Collaborator

@JLGarber JLGarber left a comment

Choose a reason for hiding this comment

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

No obvious issues after a full pass through. I haven't had time to test in-game, but if your testing has been fine, I think it's good enough. I'm confident the community will let us know quickly if there are any issues.

@github-actions github-actions bot removed the needs-review Awaiting review label Feb 22, 2025
@wexxlee wexxlee changed the title [WIP] raidboss: FRU p4 triggers raidboss: FRU p4 triggers Feb 22, 2025
@wexxlee wexxlee merged commit 30c3933 into OverlayPlugin:main Feb 22, 2025
22 checks passed
@wexxlee wexxlee deleted the fru-p4 branch February 22, 2025 21:38
github-actions bot pushed a commit that referenced this pull request Feb 22, 2025
~~Marking this as WIP for now because I want to get it posted for
feedback but may not be able to get back to it until next week. ~~

Working versions of these triggers have been tested quite a bit in game,
but I changed a few things before PRing so I'm planning to re-test in
game next week. (Probably also some typos in the comments, but I need to
look again with fresh eyes.) 30c3933
github-actions bot pushed a commit that referenced this pull request Feb 22, 2025
~~Marking this as WIP for now because I want to get it posted for
feedback but may not be able to get back to it until next week. ~~

Working versions of these triggers have been tested quite a bit in game,
but I changed a few things before PRing so I'm planning to re-test in
game next week. (Probably also some typos in the comments, but I need to
look again with fresh eyes.) 30c3933
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
raidboss /ui/raidboss module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants