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 P2 initial triggers #526

Merged
merged 7 commits into from
Dec 7, 2024

Conversation

wexxlee
Copy link
Collaborator

@wexxlee wexxlee commented Dec 2, 2024

Some early p2 triggers -- tested in emulator + vod, but hasn't yet been tested in game (although I should be able to tomorrow night unless someone else gets to it first).

@github-actions github-actions bot added raidboss /ui/raidboss module needs-review Awaiting review labels Dec 2, 2024
@github-actions github-actions bot removed the needs-review Awaiting review label Dec 2, 2024
@github-actions github-actions bot added the needs-review Awaiting review label Dec 4, 2024
@wexxlee
Copy link
Collaborator Author

wexxlee commented Dec 5, 2024

I have some more p2 stuff, but i'd like to test it a little more in-game tomorrow so I'll add it as a follow-up PR.

Copy link
Collaborator

@valarnin valarnin left a comment

Choose a reason for hiding this comment

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

I'm good with these triggers as-is now, with the adjustments made from feedback. I'd be fine with a merge here and then a PR for version bump to get something out to users, unless you think your other P2 triggers are simple/won't need much feedback.

@github-actions github-actions bot removed the needs-review Awaiting review label Dec 7, 2024
@wexxlee
Copy link
Collaborator Author

wexxlee commented Dec 7, 2024

I'm good with these triggers as-is now, with the adjustments made from feedback. I'd be fine with a merge here and then a PR for version bump to get something out to users, unless you think your other P2 triggers are simple/won't need much feedback.

Sounds good. I'll merge and version bump tomorrow morning. I have the rest of P2 done and should be able to add most or all of P3 tomorrow. But there's going to be some judgment calls/feedback for some of that, and I want to do a little more in-game testing of the P3 stuff on Monday, so probably best to leave that for a separate PR.

@wexxlee wexxlee merged commit 96dc7f2 into OverlayPlugin:main Dec 7, 2024
11 checks passed
@wexxlee wexxlee deleted the fru-p2-initial-triggers branch December 7, 2024 15:58
github-actions bot pushed a commit that referenced this pull request Dec 7, 2024
Some early p2 triggers -- tested in emulator + vod, but hasn't yet been
tested in game (although I should be able to tomorrow night unless
someone else gets to it first). 96dc7f2
github-actions bot pushed a commit that referenced this pull request Dec 7, 2024
Some early p2 triggers -- tested in emulator + vod, but hasn't yet been
tested in game (although I should be able to tomorrow night unless
someone else gets to it first). 96dc7f2
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.

4 participants