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

Feat:add missing parameter state_class #101

Merged
merged 7 commits into from
Jan 3, 2025
Merged

Conversation

DoraTiger
Copy link
Contributor

  1. add optional parameter for SPEC_PROP_TRANS_MAP, in order to add default state_class and unit:
{
    'entities':{
        '<entity name>':{
            'format': set<str>,
            'access': set<str>
        }
    },
    'properties': {
        '<property instance name>':{
            'device_class': str,
            'entity': str,
            'optional':{
                'state_class': str,
                'unit_of_measurement': str
            }
        }
    }
}
  1. Modify the relevant logic to adapt to parameter modifications.

Relation Issue: #30

@SusanPhevos
Copy link
Contributor

And #245

@SusanPhevos SusanPhevos self-requested a review December 24, 2024 12:42
@topsworld topsworld self-requested a review December 24, 2024 13:28
@iceveil
Copy link

iceveil commented Jan 3, 2025

这个内容什么时候合并呢?

@topsworld topsworld added the enhancement New feature or request label Jan 3, 2025
@topsworld topsworld merged commit 5438698 into XiaoMi:main Jan 3, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants