Skip to content

Conversation

@LakshminarayananShenbagaraj

Reason for change:
RDKB-59763 - Develop that US acceptance criteria.

Test Procedure:
Build should passed
WANManager functionality should work without any issue

Risks: Medium

Reason for change:
RDKB-59763 - Develop that US acceptance criteria.

Test Procedure:
Build should passed
WANManager functionality should work without any issue

Risks: Medium

Signed-off-by: LakshminarayananShenbagaraj <[email protected]>
@LakshminarayananShenbagaraj LakshminarayananShenbagaraj added the enhancement New feature or request label Sep 26, 2025
@LakshminarayananShenbagaraj LakshminarayananShenbagaraj marked this pull request as draft September 26, 2025 09:53
@LakshminarayananShenbagaraj LakshminarayananShenbagaraj marked this pull request as ready for review October 1, 2025 14:07
@LakshminarayananShenbagaraj LakshminarayananShenbagaraj requested a review from a team as a code owner October 17, 2025 11:02
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements WAN Manager status event handling via RBUS to trigger network response scripts based on WAN connectivity changes. It subscribes to WAN status events and executes appropriate network response actions when the WAN interface transitions to "Up" state or other conditions.

Key Changes:

  • Added RBUS event subscription for WAN Manager current status changes
  • Implemented event handler to trigger network_response.sh script based on WAN status
  • Enhanced network_response.sh to check for Hotspot interface availability in addition to LTE

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
cosa_rbus_handler_apis.h Added WAN Manager status TR181 parameter definition and function declaration for WAN status event subscription
cosa_rbus_handler_apis.c Implemented WAN status event handler and subscription logic to trigger network response scripts
network_response.sh Added Hotspot interface availability check alongside existing LTE interface check

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copilot AI review requested due to automatic review settings October 31, 2025 17:23
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings November 4, 2025 12:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings November 10, 2025 20:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 9 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings November 10, 2025 20:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings November 10, 2025 20:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings November 10, 2025 20:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants