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

model_id included in tracker dict #237

Merged
merged 9 commits into from
Feb 13, 2024
Merged

model_id included in tracker dict #237

merged 9 commits into from
Feb 13, 2024

Conversation

DigiH
Copy link
Member

@DigiH DigiH commented Feb 13, 2024

model_id included in tracker dict to allow for model_id dependant additional properties in the "presence": "absent" message when general_presence is true, but discovery is false.

Checklist:

  • I have created the pull request against the latest development branch
  • I have added only one feature/fix per PR and the code change compiles without warnings
  • I accept the Developer Certificate of Origin (DCO).

model_id included in tracker dict to allow for model_id dependant additional properties in the "presence": "absent" message when general_presence is true, but discovery is false.
@DigiH DigiH force-pushed the modelid branch 3 times, most recently from 87d7aca to ed353cb Compare February 13, 2024 17:49
@DigiH DigiH marked this pull request as draft February 13, 2024 17:50
@DigiH DigiH force-pushed the modelid branch 6 times, most recently from 43394c1 to 96bc011 Compare February 13, 2024 18:38
@DigiH DigiH marked this pull request as ready for review February 13, 2024 20:31
@DigiH DigiH merged commit 660679f into development Feb 13, 2024
7 checks passed
@DigiH DigiH deleted the modelid branch February 13, 2024 20:52
@DigiH DigiH restored the modelid branch February 13, 2024 21:22
@DigiH DigiH deleted the modelid branch February 13, 2024 21:22
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.

2 participants