Skip to content

docs: fix DPS310 availability note on MSR-2, MTR-1, AIR-1#732

Merged
bharvey88 merged 1 commit intomainfrom
docs/fix-dps310-note
Mar 19, 2026
Merged

docs: fix DPS310 availability note on MSR-2, MTR-1, AIR-1#732
bharvey88 merged 1 commit intomainfrom
docs/fix-dps310-note

Conversation

@bharvey88
Copy link
Copy Markdown
Contributor

@bharvey88 bharvey88 commented Mar 19, 2026

Summary

  • Added DPS310 note to Temperature entries (previously only on Pressure) for MSR-2 and MTR-1
  • Removed em dash from note wording
  • Updated phrasing to "if this sensor is unknown" to reflect Home Assistant entity state

Test plan

  • Verify DPS310 Pressure and Temperature entries both show the note on MSR-2 and MTR-1 pages
  • Verify both DPS310 Pressure Offset entries on AIR-1 show the updated note
  • Confirm no em dashes in rendered output

Summary by CodeRabbit

  • Documentation
    • Updated sensor definitions documentation across product guides to clarify DPS310 sensor support information and conditional notes for missing or unsupported sensors.

- Add note to DPS310 Temperature entries (was only on Pressure)
- Remove em dash from note wording
- Update phrasing to "if this sensor is unknown" to reflect HA entity state
@bharvey88 bharvey88 merged commit af9278d into main Mar 19, 2026
@bharvey88 bharvey88 deleted the docs/fix-dps310-note branch March 19, 2026 12:10
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 19, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ebba9b64-1629-4342-a815-3fdb32c9aaf5

📥 Commits

Reviewing files that changed from the base of the PR and between a7edfec and 938a313.

📒 Files selected for processing (3)
  • docs/products/air1/setup/sensor-definitions.md
  • docs/products/msr2/setup/msr2-sensor-definitions.md
  • docs/products/mtr1/setup/sensor-definitions.md

Walkthrough

Documentation updates across three product setup guides clarify conditional notes for DPS310 sensor descriptions. Phrasing about missing or unsupported sensors was revised to use consistent language stating "If this sensor/option is unknown" instead of variations of "if you do not see this."

Changes

Cohort / File(s) Summary
DPS310 Sensor Documentation Updates
docs/products/air1/setup/sensor-definitions.md, docs/products/msr2/setup/msr2-sensor-definitions.md, docs/products/mtr1/setup/sensor-definitions.md
Standardized conditional note phrasing for DPS310 Pressure and DPS310 Temperature sensor descriptions. Replaced "if you do not see this..." variants with "If this sensor/option is unknown" to clarify device support conditions across AIR-1, MSR-2, and MTR-1 product documentation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

A clarifying hop through sensor notes so true,
Where DPS310 finds its clearer view,
When readings hide, the words now say with grace,
"If unknown, check devices in their place" 🐰✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/fix-dps310-note
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant