zwave-js / zwave-js-ui / 22177442992

19%

push

github

feat: add individual value polling button to UI (#4525)

## Add Manual Poll Value Button to UI

This PR implements the feature to manually poll individual values from
the UI as requested in the issue.

### Changes Made
- [x] Explore codebase and understand architecture
- [x] Create reusable PollValueButton component
- [x] Created dedicated `PollValueButton.vue` component to eliminate
code duplication
- [x] Encapsulates pollValue API call logic and loading state management
  - [x] Handles success/error responses with snackbar notifications
  - [x] Try-finally block for proper error handling
- [x] Add poll button to all values in ValueId.vue
  - [x] Import and use PollValueButton component
  - [x] Added to read-only values (plain text display)
- [x] Added to all writable input types using `append-inner` slot (text,
number, select, combobox, duration, color)
  - [x] Added to boolean button groups
- [x] Only shows for readable values via canPollValue computed property
- [x] Address PR feedback
- [x] Refactored to use `append-inner` slot instead of separate buttons
  - [x] Created dedicated component to eliminate ALL code duplication
- [x] Added button to ALL inputs including read-only values (previously
missing)
  - [x] Button appears consistently across all value types
- [x] Test the implementation
  - [x] Manually tested with mock Z-Wave controller
  - [x] Verified button appears on read-only values
  - [x] Verified button appears on all writable input types
  - [x] Verified polling works correctly
  - [x] Screenshots captured showing UI changes
- [x] Run linting and tests
  - [x] Run lint-fix and lint (passed)
  - [x] Run test suite (all 103 tests passed)
  - [x] Build succeeded

### Technical Details
- **Backend**: pollValue API already exists in ZwaveClient.ts
- **Frontend**: Created reusable `PollValueButton.vue` component
- **Communication**: Uses app.apiRequest('pollValue', [valueId])
- **UI**: Small icon button with loading spinner, appears consi... (continued)

334 of 444 branches covered (75.23%)

Branch coverage included in aggregate %.

4052 of 22316 relevant lines covered (18.16%)

1.13 hits per line