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

raidemulator: Fix trigger popup text not showing up #556

Merged

Conversation

Huynhs
Copy link

@Huynhs Huynhs commented Jan 6, 2025

For timeline triggers, alert text / tts was not showing up in raidemulator.

Due to circular dependency in bindTo, a setter was added for popup text. Suggestions welcome since I am not familiar with typescript / this project

@cactbotbot
Copy link
Collaborator

cactbotbot commented Jan 6, 2025

@Huynhs Thanks for your contribution! 🌵🚀

@github-actions github-actions bot added needs-review Awaiting review raidemulator /ui/raidemulator module labels Jan 6, 2025
@Huynhs Huynhs changed the title raidemulator: Fix popup text not showing up in raid emulator for some triggers raidemulator: Fix trigger popup text not showing up Jan 6, 2025
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.

Thanks for taking the time to fix this! If you don't have time to address the suggestions below, indicate as such and someone can change them in a follow up PR.

@github-actions github-actions bot removed the needs-review Awaiting review label Jan 12, 2025
@Huynhs
Copy link
Author

Huynhs commented Jan 13, 2025

Applied suggestions, let me know if anything else looks off

@valarnin valarnin merged commit db3f621 into OverlayPlugin:main Jan 13, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
raidemulator /ui/raidemulator module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants