Skip to content

Conversation

Fmstrat
Copy link

@Fmstrat Fmstrat commented Jul 25, 2025

Description:

This PR updates the documentation for the NOX sensor to align to the actual specs in the datasheet, vs copying the non-matching VOC defaults. It also:

  • Adds NOX to the sample YAML
  • Removes ranges where ranges are not provided in the datasheet (even if they are similar sensors)

Related issue (if applicable): fixes esphome/issues#6330

Pull request in esphome with YAML changes (if applicable):

Checklist:

  • I am merging into next because this is new documentation that has a matching pull-request in esphome as linked above.
    or

  • I am merging into current because this is a fix, change and/or adjustment in the current documentation and is not for a new component or feature.

  • Link added in /components/index.rst when creating new documents for new components or cookbook.

Copy link

netlify bot commented Jul 25, 2025

Deploy Preview for esphome ready!

Name Link
🔨 Latest commit 3927de6
🔍 Latest deploy log https://app.netlify.com/projects/esphome/deploys/68b108918738b30008921ff8
😎 Deploy Preview https://deploy-preview-5176--esphome.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 project configuration.

@jesserockz jesserockz changed the title Adjust sen5x to match VOC/NOX datasheet [sen5x] Update Nox default and lint page Aug 29, 2025
@jesserockz jesserockz changed the title [sen5x] Update Nox default and lint page [sen5x] Update Nox defaults and lint page Aug 29, 2025
@jesserockz jesserockz merged commit e7d5b6e into esphome:next Aug 29, 2025
7 of 8 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Aug 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants