fix: fix refresh power button issue by DevipriyaS17 · Pull Request #3441 · device-management-toolkit/sample-web-ui

DevipriyaS17 · 2026-07-14T08:39:56Z

PR Checklist

Unit Tests have been added for new changes
API tests have been updated if applicable
All commented code has been removed
If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Issue - #3440 Clicking the Refresh Power Status icon does not retrieve the latest power state

Root Cause:

The toolbar was using getPowerStateCached() for power-state reads, which can return a previously cached value instead of issuing a new HTTP request. That is correct for the initial load, but not for a manual refresh.

Behavior:

User opens the device detail page
ngOnInit() loads the current power state using the cached path
Toolbar shows the current state
User clicks the Refresh Power Status icon
The refresh action now bypasses the cache and calls devicesService.getPowerState()
The UI receives the latest state from the backend instead of reusing stale cached data

Fix:

Kept the initial load path using getPowerStateCached() so simultaneous startup requests remain deduped
Changed the refresh action to call getPowerState() directly so every manual refresh makes a fresh HTTP request
Updated the refresh control to use a real button for keyboard accessibility
Added/updated tests to cover the refresh path and preserve the cached-vs-fresh behavior

Request URL
https://10.99.115.56/mps/api/v1/amt/power/state/4939adac-fc23-8c93-c2d0-88aedd6e4316
Request Method
GET
Status Code
200 OK
{"powerstate":2,"OSPowerSavingState":2}

Request URL
https://10.99.115.56/mps/api/v1/amt/power/action/4939adac-fc23-8c93-c2d0-88aedd6e4316
Request Method
POST
Status Code
200 OK
{
"Header": {
"To": "http://schemas.xmlsoap.org/ws/2004/08/addressing/role/anonymous",
"RelatesTo": 0,
"Action": "http://schemas.dmtf.org/wbem/wscim/1/cim-schema/2/CIM_PowerManagementService/RequestPowerStateChangeResponse",
"MessageID": "uuid:00000000-8086-8086-8086-00000000014C",
"ResourceURI": "http://schemas.dmtf.org/wbem/wscim/1/cim-schema/2/CIM_PowerManagementService"
},
"Body": {
"ReturnValue": 0,
"ReturnValueStr": "SUCCESS"
}
}

3.Request URL
https://10.99.115.56/mps/api/v1/amt/power/state/4939adac-fc23-8c93-c2d0-88aedd6e4316
Request Method
GET
Status Code
200 OK
Remote Address

{"powerstate":8,"OSPowerSavingState":0}

Copilot

Pull request overview

This PR fixes a regression on the Device Details toolbar where clicking the Refresh Power Status icon could return a stale cached power state instead of fetching the latest state from the device (issue #3440).

Changes:

Update the toolbar power-state refresh flow to call devicesService.getPowerState() (fresh HTTP request) instead of getPowerStateCached().
Update the inline comment to reflect the intended “bypass cache on refresh” behavior.

Copilot

Pull request overview

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

Copilot

Pull request overview

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

Copilot

Pull request overview

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

Copilot

Pull request overview

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

Copilot

Pull request overview

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

Copilot

Pull request overview

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

Copilot

Pull request overview

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

Copilot

Pull request overview

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

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

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

madhavilosetty-intel · 2026-07-15T17:57:12Z

@DevipriyaS17

When I power off the device, though I get the response as {ReturnValue: 0}, Power on button is green until I refresh the page. Same thing happen when I power up the device.

I have tested on AMT 15 and AMT 19

DevipriyaS17 · 2026-07-16T08:55:17Z

@DevipriyaS17

When I power off the device, though I get the response as {ReturnValue: 0}, Power on button is green until I refresh the page. Same thing happen when I power up the device.

I have tested on AMT 15 and AMT 19

Updated the code to fix the issue

## [3.60.1](v3.60.0...v3.60.1) (2026-07-20) ### Bug Fixes * fix refresh power button issue ([#3441](#3441)) ([98210d2](98210d2))

RosieAMT · 2026-07-20T05:11:51Z

🎉 This PR is included in version 3.60.1 🎉

The release is available on:

GitHub release
v3.60.1

Your semantic-release bot 📦🚀

DevipriyaS17 requested a review from Copilot July 14, 2026 08:40

Copilot started reviewing on behalf of DevipriyaS17 July 14, 2026 08:40 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

Comment thread src/app/devices/device-toolbar/device-toolbar.component.ts Outdated

Comment thread src/app/devices/device-toolbar/device-toolbar.component.ts Outdated

DevipriyaS17 mentioned this pull request Jul 14, 2026

fix: fix refresh power button issue. #3430

Closed

4 tasks

DevipriyaS17 force-pushed the pr-3430 branch from b9aab4b to cca39b4 Compare July 14, 2026 08:53

DevipriyaS17 requested a review from Copilot July 14, 2026 08:58

Copilot started reviewing on behalf of DevipriyaS17 July 14, 2026 08:58 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

Comment thread src/app/devices/device-toolbar/device-toolbar.component.ts Outdated

DevipriyaS17 force-pushed the pr-3430 branch from f78e2d3 to ee3ae1b Compare July 14, 2026 09:01

DevipriyaS17 requested a review from Copilot July 14, 2026 09:03

Copilot started reviewing on behalf of DevipriyaS17 July 14, 2026 09:03 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

Comment thread src/app/devices/device-toolbar/device-toolbar.component.ts Outdated

Comment thread src/app/devices/device-toolbar/device-toolbar.component.ts

DevipriyaS17 force-pushed the pr-3430 branch from ee3ae1b to 8f8c40e Compare July 14, 2026 09:14

DevipriyaS17 requested a review from Copilot July 14, 2026 09:15

Copilot started reviewing on behalf of DevipriyaS17 July 14, 2026 09:15 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

Comment thread src/app/devices/device-toolbar/device-toolbar.component.ts Outdated

DevipriyaS17 force-pushed the pr-3430 branch from 8f8c40e to 0e17ac2 Compare July 14, 2026 09:22

DevipriyaS17 requested a review from Copilot July 14, 2026 09:22

Copilot started reviewing on behalf of DevipriyaS17 July 14, 2026 09:22 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

Comment thread src/app/devices/device-toolbar/device-toolbar.component.html Outdated

Comment thread src/app/devices/device-toolbar/device-toolbar.component.ts Outdated

DevipriyaS17 force-pushed the pr-3430 branch from 0e17ac2 to 645fca0 Compare July 15, 2026 05:16

DevipriyaS17 requested a review from Copilot July 15, 2026 05:16

Copilot started reviewing on behalf of DevipriyaS17 July 15, 2026 05:16 View session

Copilot AI reviewed Jul 15, 2026

View reviewed changes

Comment thread src/app/devices/device-toolbar/device-toolbar.component.ts

Comment thread src/app/devices/device-toolbar/device-toolbar.component.ts Outdated

DevipriyaS17 force-pushed the pr-3430 branch from 9f108c5 to fa4a220 Compare July 15, 2026 05:35

DevipriyaS17 requested a review from Copilot July 15, 2026 05:36

Copilot started reviewing on behalf of DevipriyaS17 July 15, 2026 05:36 View session

punam20 and others added 2 commits July 15, 2026 11:06

fix: fix refresh power button issue

86f4a60

fix: address copilot review comments

919af53

Copilot AI reviewed Jul 15, 2026

View reviewed changes

Comment thread src/app/devices/device-toolbar/device-toolbar.component.html Outdated

Copilot AI reviewed Jul 15, 2026

View reviewed changes

Comment thread src/app/devices/device-toolbar/device-toolbar.component.spec.ts

DevipriyaS17 requested a review from Copilot July 15, 2026 06:00

Copilot started reviewing on behalf of DevipriyaS17 July 15, 2026 06:01 View session

Copilot AI reviewed Jul 15, 2026

View reviewed changes

Comment thread src/app/devices/device-toolbar/device-toolbar.component.ts

DevipriyaS17 force-pushed the pr-3430 branch from ba16757 to e92baf3 Compare July 15, 2026 06:10

DevipriyaS17 requested a review from Copilot July 15, 2026 06:11

Copilot started reviewing on behalf of DevipriyaS17 July 15, 2026 06:11 View session

fix: fix for copilot comment

e92baf3

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI reviewed Jul 15, 2026

View reviewed changes

DevipriyaS17 marked this pull request as ready for review July 15, 2026 06:36

DevipriyaS17 requested review from graikhel-intel and madhavilosetty-intel July 15, 2026 06:38

madhavilosetty-intel reviewed Jul 15, 2026

View reviewed changes

Comment thread src/app/devices/device-toolbar/device-toolbar.component.html Outdated

madhavilosetty-intel reviewed Jul 15, 2026

View reviewed changes

Comment thread src/app/devices/device-toolbar/device-toolbar.component.ts

Merge branch 'main' into pr-3430

f9a4bf8

DevipriyaS17 force-pushed the pr-3430 branch 2 times, most recently from b342e39 to 755c277 Compare July 16, 2026 08:46

DevipriyaS17 requested a review from madhavilosetty-intel July 16, 2026 08:55

fix: address review comments

6f07ed6

madhavilosetty-intel force-pushed the pr-3430 branch from 755c277 to 6f07ed6 Compare July 16, 2026 20:04

madhavilosetty-intel approved these changes Jul 16, 2026

View reviewed changes

Merge branch 'main' into pr-3430

5b2794a

DevipriyaS17 merged commit 98210d2 into main Jul 20, 2026
10 checks passed

DevipriyaS17 deleted the pr-3430 branch July 20, 2026 04:59

RosieAMT pushed a commit that referenced this pull request Jul 20, 2026

chore(release): 3.60.1 [skip ci]

ac5e83e

## [3.60.1](v3.60.0...v3.60.1) (2026-07-20) ### Bug Fixes * fix refresh power button issue ([#3441](#3441)) ([98210d2](98210d2))

RosieAMT added the released label Jul 20, 2026

DevipriyaS17 mentioned this pull request Jul 20, 2026

refactor: add refresh power status button label key to all languages #3448

Merged

4 tasks

Uh oh!

Conversation

DevipriyaS17 commented Jul 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Checklist

What are you changing?

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

madhavilosetty-intel commented Jul 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

DevipriyaS17 commented Jul 16, 2026

Uh oh!

Uh oh!

RosieAMT commented Jul 20, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

DevipriyaS17 commented Jul 14, 2026 •

edited

Loading

madhavilosetty-intel commented Jul 15, 2026 •

edited

Loading