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.

Sign up for GitHub

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

Jump to bottom

[1.3] DRLC cluster is in the wrong domain in ZAP (should be Energy Management, not Energy) #31690

Closed
jamesharrow opened this issue Jan 25, 2024 · 0 comments · Fixed by #31692
Closed

[1.3] DRLC cluster is in the wrong domain in ZAP (should be Energy Management, not Energy) #31690

jamesharrow opened this issue Jan 25, 2024 · 0 comments · Fixed by #31692
Assignees
@jamesharrow
Labels
app-clusters Application cluster work bug Something isn't working energy needs triage

Comments

@jamesharrow
Copy link
Contributor

Reproduction steps

  1. Open ZAP
  2. Look and see that DRLC is in the wrong place compared to other clusters (should be Energy Management per spec).

Before this gets left, it is important to clear this up despite DRLC not being in 1.3.

Simple fix to change the 'domain' in XML.

Bug prevalence

Every time

GitHub hash of the SDK that was being used

e5f5d67

Platform

core

Platform Version(s)

No response

Type

Spec Compliance Issue

Anything else?

No response
@jamesharrow jamesharrow added bug Something isn't working needs triage labels Jan 25, 2024
@jamesharrow jamesharrow self-assigned this Jan 25, 2024
@jamesharrow jamesharrow added app-clusters Application cluster work energy labels Jan 25, 2024
jamesharrow added a commit to jamesharrow/connectedhomeip that referenced this issue Jan 25, 2024
@jamesharrow
Fixes project-chip#31690 Moved provisional DRLC cluster into Energy M…
1a0a9e3 
…anagement domain so it shows in ZAP in the correct place.
@jamesharrow jamesharrow mentioned this issue Jan 25, 2024
Merged
@mergify mergify bot closed this as completed in #31692 Feb 2, 2024
mergify bot pushed a commit that referenced this issue Feb 2, 2024
@jamesharrow
Fixes #31690 Moved provisional DRLC cluster into Energy Management do…
1441d62 
…main so it shows in ZAP in the correct place. (#31692)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jamesharrow jamesharrow

Labels
app-clusters Application cluster work bug Something isn't working energy needs triage
Projects
[Grouping] Energy Management
Archived in project
Milestone
No milestone
1 participant
@jamesharrow