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

[SONOFF-POWCT] Update config to align with Esphome PR#7836 #1007

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mick96
Copy link

@mick96 mick96 commented Jan 29, 2025

Enhance SONOFF-POWCT configuration with CSE7761 integration improvements from esphome/esphome#7836.

Brief description of the changes

Type of changes

  • New device
  • Update existing device
  • Removing a device
  • General cleanup
  • Other

Checklist:

  • There are no passwords or secrets references in any examples.
    The only exceptions are !secret wifi_ssid and !secret wifi_password.
  • The wifi or ethernet block has no static / manual ip address specified.
  • The first configuration provided should be hardware definitions only.
    A more involved example can be provided in a separate configuration block.

The PR fixes cse7761 compatibility with SONOFF-POWCT and support for frequency.
Copy link

netlify bot commented Jan 29, 2025

Deploy Preview for esphome-devices ready!

Name Link
🔨 Latest commit bc5f2e5
🔍 Latest deploy log https://app.netlify.com/sites/esphome-devices/deploys/679aa1c708b1f50008e29cb5
😎 Deploy Preview https://deploy-preview-1007--esphome-devices.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 92 (🔴 down 1 from production)
Accessibility: 86 (no change from production)
Best Practices: 100 (no change from production)
SEO: 89 (no change from production)
PWA: 70 (no change from production)
View the detailed breakdown and full score reports

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

@h1771t
Copy link

h1771t commented Jan 30, 2025

Thanks for this. The old line 96 should also be removed no?
Anyway with the filters removed, and the patches from the PR mentioned, this config is working very well for me on the POWCT.

@jesserockz jesserockz marked this pull request as draft February 13, 2025 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants