-
-
Notifications
You must be signed in to change notification settings - Fork 2k
[18.0][IMP] web_ir_actions_act_window_message: add demo action #3353
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
base: 18.0
Are you sure you want to change the base?
[18.0][IMP] web_ir_actions_act_window_message: add demo action #3353
Conversation
StefanRijnhart
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Do you want to mention this demo functionality somewhere in the readme?
956a149 to
5f164f5
Compare
|
thanks @StefanRijnhart, added that and while being on it, fixed some issues with the readme |
StefanRijnhart
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
sergioM-S73
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
@hbrunn Please check the conflict in |
this possible now already with web_notify
827a672 to
e8cda0b
Compare
|
@StefanRijnhart thanks for the heads-up, done |
this should help to avoid botched migrations by showing contributors and reviewers what the expected functionality is