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

[Sitemaps] Remove parameter sendFrequency #2346

Merged
merged 1 commit into from
Aug 18, 2024

Conversation

mueller-ma
Copy link
Member

This parameter has been used in Classic UI which isn't available anymore.

I also checked the Android app and it isn't used there.

This parameter has been used in Classic UI which isn't available anymore.

I also checked the Android app and it isn't used there.

Signed-off-by: mueller-ma <mueller-ma@users.noreply.github.com>
Copy link

netlify bot commented Aug 7, 2024

Thanks for your pull request to the openHAB documentation! The result can be previewed at the URL below (this comment and the preview will be updated if you add more commits).

Built without sensitive environment variables

Name Link
🔨 Latest commit 7b49fd9
🔍 Latest deploy log https://app.netlify.com/sites/openhab-docs-preview/deploys/66b3a4e55c5293000875f73e
😎 Deploy Preview https://deploy-preview-2346--openhab-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@stefan-hoehn
Copy link
Contributor

@mueller-ma are you sure? I checked the webui-project and I can still find it in /src/components/pagedesigner/sitemap for slider and colorpicker

@mueller-ma
Copy link
Member Author

mueller-ma commented Aug 12, 2024

Yes, I'm sure. See the linked issue for more informatiom about the removal of this parameter.

openhab/openhab-core#4338

@stefan-hoehn
Copy link
Contributor

And finally @florian-h05 also removed the respective part in webui with #2705

@stefan-hoehn stefan-hoehn merged commit 3fae76f into openhab:main Aug 18, 2024
4 checks passed
@mueller-ma mueller-ma deleted the patch-5 branch August 19, 2024 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants