Skip to content

RDKCOM-5592: RDKBDEV-3444, RDKBACCL-1608: [TDK][AUTO][BPI][DML]Device.WiFi.NeighboringWiFiDiagnostic.DiagnosticsState is empty#1118

Open
ksaipr036 wants to merge 1 commit into
rdkcentral:developfrom
ksaipr036:rdkbaccl-1608
Open

RDKCOM-5592: RDKBDEV-3444, RDKBACCL-1608: [TDK][AUTO][BPI][DML]Device.WiFi.NeighboringWiFiDiagnostic.DiagnosticsState is empty#1118
ksaipr036 wants to merge 1 commit into
rdkcentral:developfrom
ksaipr036:rdkbaccl-1608

Conversation

@ksaipr036
Copy link
Copy Markdown
Contributor

Reason for Change: Set default value of DiagnosticsState to "None" instead of leaving it empty.
Test Procedure: Verify that DiagnosticsState is "None" by default when not explicitly set and it's functionality should work.
Risks: Low.

@ksaipr036 ksaipr036 marked this pull request as ready for review May 21, 2026 06:27
@ksaipr036 ksaipr036 requested a review from a team as a code owner May 21, 2026 06:27
@ksaipr036 ksaipr036 changed the title RDKBACCL-1608: [TDK][AUTO][BPI][DML]Device.WiFi.NeighboringWiFiDiagnostic.DiagnosticsState is empty RDKBDEV-3444, RDKBACCL-1608: [TDK][AUTO][BPI][DML]Device.WiFi.NeighboringWiFiDiagnostic.DiagnosticsState is empty May 21, 2026
@ksaipr036 ksaipr036 force-pushed the rdkbaccl-1608 branch 2 times, most recently from 61fc3d5 to fa2f66b Compare May 28, 2026 10:54
@manigandanj
Copy link
Copy Markdown
Contributor

Looks good to me.

…stic.DiagnosticsState is empty

Reason for Change: Set default value of DiagnosticsState to "None" instead of leaving it empty.
Test Procedure: Verify that DiagnosticsState is "None" by default when not explicitly set and it's functionality should work.
Risks: Low.

Signed-off-by: ksaipr036 <kosika_saipriya@comcast.com>
@pradeeptakdas pradeeptakdas changed the title RDKBDEV-3444, RDKBACCL-1608: [TDK][AUTO][BPI][DML]Device.WiFi.NeighboringWiFiDiagnostic.DiagnosticsState is empty RDKCOM-5592: RDKBDEV-3444, RDKBACCL-1608: [TDK][AUTO][BPI][DML]Device.WiFi.NeighboringWiFiDiagnostic.DiagnosticsState is empty Jun 3, 2026
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