Skip to content

MATTER-5952 : Fan control app refactor#804

Open
sarthak shaha (Sarthak-Shaha) wants to merge 14 commits into
release_2.9-1.6from
project_upgrades_fan_control_app
Open

MATTER-5952 : Fan control app refactor#804
sarthak shaha (Sarthak-Shaha) wants to merge 14 commits into
release_2.9-1.6from
project_upgrades_fan_control_app

Conversation

@Sarthak-Shaha

@Sarthak-Shaha sarthak shaha (Sarthak-Shaha) commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Issue Link:
MATTER-5952
Design doc: https://confluence.silabs.com/spaces/MATTER/pages/848763946/Fan+Control+App+Refactor

Description of Problem/Feature:

Description of Fix/Solution:
Fan Control App refactor to
matter_sdk changes:

  1. Merge datamodelcallbacks, fanControlmanager into apptask
  2. Streamline APIs
  3. Apply CRTP , ( base class, Impl and customer override from common)

Matter_extension

  1. Use the sample app component to refer to the apptask to regenerate for each upgrade
  2. Created FanControlConfig to configure endpoints and fan speed.
  3. Using the sample app component to refer to the config file

Testing Done:
CI
Smoke: https://jenkins-cbs-sqa.silabs.net/blue/organizations/jenkins/sqa-pipelines-matter/detail/project_upgrades_fan_control_app%2Fsmoke%2Fmatter/2/tests no new failures

@Sarthak-Shaha sarthak shaha (Sarthak-Shaha) changed the title Initial Commit Fan control refactor MATTER-5952 : Fan control app refactor Jun 3, 2026
@Sarthak-Shaha sarthak shaha (Sarthak-Shaha) marked this pull request as ready for review June 4, 2026 15:24
@Sarthak-Shaha sarthak shaha (Sarthak-Shaha) requested a review from a team as a code owner June 4, 2026 15:24
@Sarthak-Shaha sarthak shaha (Sarthak-Shaha) added the sl-require-admin-action This TAG indicates that an admin must take action. label Jun 8, 2026
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

The check for sl-require-admin-action label CI failure for this job is normal. An admin must do the merge.

@@ -212,7 +212,7 @@ pipeline
stage('Trigger SQA Smoke Pipeline')

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DO NOT MERGE BEFORE v2.9.0 is released.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sl-require-admin-action This TAG indicates that an admin must take action.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant