Skip to content

RDKEAPPRT-844,843 My Apps tile icons update offline/online state only after reboot when network is disconnected or reconnected.,Resolution and HDR displaying the technical value instead of User known values #215

Merged
suryag23 merged 7 commits into
developfrom
844
Jun 26, 2026

Conversation

@suryag23

Copy link
Copy Markdown
Contributor

No description provided.

suryag23 added 2 commits June 25, 2026 13:04
…nstead of User known values ,My Apps tile icons update offline/online state only after reboot when network is disconnected or reconnected.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates RefUI to better reflect network connectivity changes in app icon rendering and to display user-friendly values for video settings (resolution/HDR), along with a version bump.

Changes:

  • Bump RefUI version from 6.0.20 to 6.0.21 (Bolt package config + UI settings).
  • Update MainView to refresh My Apps/DAC rows on connectivity changes and show offline placeholders when disconnected.
  • Remove technical FireBolt-derived resolution/HDR/HDCP strings in favor of AppApi-provided, user-facing values (Video screen + overlay).

Reviewed changes

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

Show a summary per file
File Description
bolt/package-configs/com.rdkcentral.refui.json Version bump to 6.0.21 for the Bolt package config.
accelerator-home-ui/src/views/MainView.js Refresh My Apps/DAC rows on internet status change; add My Apps offline placeholder support.
accelerator-home-ui/src/views/AppInfoPage.js Add internet status handling to toggle offline placeholders for installed app cards.
accelerator-home-ui/src/screens/VideoAndAudioScreens/VideoScreen.js Use AppApi values for resolution/HDR/HDCP instead of technical FireBolt output.
accelerator-home-ui/src/overlays/AudioScreens/VideoScreenOverlay.js Same as VideoScreen, for overlay view.
accelerator-home-ui/src/items/AppCard.js Add offline placeholder image handling for app icons.
accelerator-home-ui/settings.json Version bump to 6.0.21 for UI settings.

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

Comment thread accelerator-home-ui/src/views/MainView.js Outdated
Comment thread accelerator-home-ui/src/views/AppInfoPage.js
Comment thread accelerator-home-ui/src/views/AppInfoPage.js Outdated

Copilot AI left a comment

Copy link
Copy Markdown

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 7 out of 7 changed files in this pull request and generated 4 comments.

Comments suppressed due to low confidence (2)

accelerator-home-ui/src/screens/VideoAndAudioScreens/VideoScreen.js:199

  • After removing the non-ResidentApp FireBoltApi/GLOBALS branch in _focus(), the FireBoltApi and GLOBALS imports at the top of this file appear to be unused and should be removed to avoid dead code / linter failures.
  _focus() {
      this._appApi.getResolution().then(resolution => {
        this.tag("Resolution.Title").text.text = Language.translate('Resolution: ') + resolution;
      }).catch(err => {
        this.ERR("Error fetching the Resolution: " + JSON.stringify(err))

accelerator-home-ui/src/overlays/AudioScreens/VideoScreenOverlay.js:192

  • FireBoltApi and GLOBALS are no longer referenced in this overlay after removing the non-ResidentApp branch in _focus(). Please remove the unused imports at the top of the file to avoid dead code / potential linter failures.
  _focus() {
      this._appApi.getResolution().then(resolution => {
        this.tag("Resolution.Title").text.text = Language.translate('Resolution: ') + resolution;
      }).catch(err => {
        this.ERR("Error fetching the Resolution" + JSON.stringify(err))
      })

Comment thread accelerator-home-ui/src/views/MainView.js Outdated
Comment thread accelerator-home-ui/src/views/AppInfoPage.js
Comment thread accelerator-home-ui/src/items/AppCard.js
Comment thread accelerator-home-ui/src/views/AppInfoPage.js

Copilot AI left a comment

Copy link
Copy Markdown

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 7 out of 7 changed files in this pull request and generated 4 comments.

Comments suppressed due to low confidence (1)

accelerator-home-ui/src/overlays/AudioScreens/VideoScreenOverlay.js:192

  • After removing the FireBoltApi / GLOBALS.selfClientName branch from _focus(), the GLOBALS and FireBoltApi imports at the top of this file are now unused, which can trigger eslint unused-import failures. Please remove those unused imports.
  _focus() {
      this._appApi.getResolution().then(resolution => {
        this.tag("Resolution.Title").text.text = Language.translate('Resolution: ') + resolution;
      }).catch(err => {
        this.ERR("Error fetching the Resolution" + JSON.stringify(err))
      })

Comment thread accelerator-home-ui/src/views/MainView.js Outdated
Comment thread accelerator-home-ui/src/views/MainView.js Outdated
Comment thread accelerator-home-ui/src/views/AppInfoPage.js
Comment thread accelerator-home-ui/src/screens/VideoAndAudioScreens/VideoScreen.js

Copilot AI left a comment

Copy link
Copy Markdown

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 7 out of 7 changed files in this pull request and generated 2 comments.

Comment thread accelerator-home-ui/src/screens/VideoAndAudioScreens/VideoScreen.js
Comment thread accelerator-home-ui/src/overlays/AudioScreens/VideoScreenOverlay.js

Copilot AI left a comment

Copy link
Copy Markdown

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 7 out of 7 changed files in this pull request and generated 3 comments.

Comment thread accelerator-home-ui/src/screens/VideoAndAudioScreens/VideoScreen.js
Comment thread accelerator-home-ui/src/overlays/AudioScreens/VideoScreenOverlay.js
Comment thread accelerator-home-ui/src/views/MainView.js

Copilot AI left a comment

Copy link
Copy Markdown

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 7 out of 7 changed files in this pull request and generated no new comments.

@yashaswini-rk yashaswini-rk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good to me

@suryag23 suryag23 merged commit 30d0560 into develop Jun 26, 2026
8 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants