-
Notifications
You must be signed in to change notification settings - Fork 817
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
Added hiding remotePollInterval selector in settings if notify push available #7941
Conversation
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.
Looks great! Other than a nitpick I think this is good to go
Hi! I implemented both of your requests, @nilsding and @claucambra, hope its good now. |
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 again :)
/backport to stable-3.16 |
… notify push is available Signed-off-by: Pablo Ariño Muñoz <[email protected]>
Signed-off-by: Pablo Ariño Muñoz <[email protected]>
00b54b3
to
c1a5eaa
Compare
Changed the set properties to the designer file and added a way for the remotePollInterval selector to hide in the settings if notify push is available. This is a follow up to #7926