Add quality scale for airnow integration #169709
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,65 @@ | ||
| rules: | ||
|
Member
There was a problem hiding this comment. Another sidenote, a lot of the variables defined in
Member
There was a problem hiding this comment. I think we can consider removing that entity/device removal |
||
| # Bronze | ||
| action-setup: | ||
| status: exempt | ||
| comment: Integration does not register custom actions. | ||
| appropriate-polling: | ||
| status: done | ||
| comment: Reports are polled every 30 minutes so newly published hourly AirNow reports are picked up promptly. | ||
| brands: done | ||
| common-modules: done | ||
| config-flow-test-coverage: done | ||
|
Member
There was a problem hiding this comment.
That test should also test the unique id of the created entry
Member
There was a problem hiding this comment. In general I would recommend to reuse
Member
There was a problem hiding this comment. For |
||
| config-flow: done | ||
|
Member
There was a problem hiding this comment. Please revert the strings from this PR and set this to todo |
||
| dependency-transparency: done | ||
| docs-actions: | ||
| status: exempt | ||
| comment: Integration does not register custom actions. | ||
| docs-high-level-description: done | ||
| docs-installation-instructions: done | ||
| docs-removal-instructions: done | ||
| entity-event-setup: | ||
| status: exempt | ||
| comment: Integration does not subscribe to events. | ||
| entity-unique-id: done | ||
| has-entity-name: done | ||
| runtime-data: done | ||
| test-before-configure: done | ||
| test-before-setup: done | ||
| unique-config-entry: done | ||
|
|
||
| # Silver | ||
| action-exceptions: todo | ||
|
|
||
| config-entry-unloading: done | ||
| docs-configuration-parameters: todo | ||
| docs-installation-parameters: todo | ||
| entity-unavailable: todo | ||
| integration-owner: todo | ||
|
|
||
| log-when-unavailable: todo | ||
| parallel-updates: done | ||
| reauthentication-flow: todo | ||
| test-coverage: todo | ||
|
Member
There was a problem hiding this comment. Sidenote, we should try and patch the library where we use it, instead of patching internals like
Member
There was a problem hiding this comment. Why do we fake the timezone in the diagnostic test? |
||
| # Gold | ||
| devices: todo | ||
|
Comment on lines
+38
to
+41
|
||
| diagnostics: done | ||
| discovery: todo | ||
| discovery-update-info: todo | ||
| docs-data-update: todo | ||
| docs-examples: todo | ||
| docs-known-limitations: todo | ||
| docs-supported-devices: todo | ||
| docs-supported-functions: todo | ||
| docs-troubleshooting: todo | ||
| docs-use-cases: todo | ||
| dynamic-devices: todo | ||
| entity-category: todo | ||
| entity-device-class: todo | ||
|
Member
There was a problem hiding this comment. Let's add to the comment in the file that you can still use ozone
Member
There was a problem hiding this comment. I am wondering if we can split the extra state attributes for AQI into separate entities in the future |
||
| entity-disabled-by-default: todo | ||
| entity-translations: todo | ||
| exception-translations: todo | ||
| icon-translations: todo | ||
| reconfiguration-flow: todo | ||
| stale-devices: todo | ||
| repair-issues: todo | ||
| # Platinum | ||
| async-dependency: todo | ||
|
|
||
| inject-websession: todo | ||
|
|
||
| strict-typing: todo | ||
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.
As a sidenote, maybe it'd make sense to migrate it to use subentries, like for example
waqi. But this is optional, just wanted to put it out there