-
Notifications
You must be signed in to change notification settings - Fork 103
RDKB-58244: SMApp update based on status of the Mesh.Enable and HCM #175
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
Conversation
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.
few modification requested over call
I can't see the comments |
710da57
to
396b90f
Compare
271c4be
to
4736e4e
Compare
@HollowMax |
4736e4e
to
58d34c4
Compare
Hi @gsathish86 resolved the conflicts, updated the code base, also Viviane's comments were addressed. |
84a5d4d
to
0e227e6
Compare
All contributors have signed the CLA ✍️ ✅ |
0e227e6
to
dadc0a0
Compare
recheck |
I have read the CLA Document and I hereby sign the CLA |
@gsathish86 can you please tell me if I need to do something else for merge? |
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.
Please address the comments.
Reason for change: SMApp continues to run when Mesh is disabled. Test Procedure: 1. Disable getv Device.DeviceInfo.X_RDKCENTRAL-COM_xOpsDeviceMgmt.Mesh.Enable and Device.DeviceInfo.X_RDKCENTRAL-COM_RFC.Feature.MeshWifiOptimization.Mode 2. Check if SM_APP is still running. Risks: Low Priority: P2 Signed-off-by: Maksym Holovatyi <[email protected]>
3e299ff
to
192564c
Compare
Reason for change: SMApp continues to run when Mesh is disabled.
Test Procedure:
Risks: Low
Priority: P2