Skip to content

Add RFID token management actions to Peblar docs#45015

Open
syphernl wants to merge 6 commits intohome-assistant:nextfrom
syphernl:peblar-rfid-token-services
Open

Add RFID token management actions to Peblar docs#45015
syphernl wants to merge 6 commits intohome-assistant:nextfrom
syphernl:peblar-rfid-token-services

Conversation

@syphernl
Copy link
Copy Markdown
Contributor

Proposed change

Adds documentation for the RFID services:

  • peblar.list_rfid_tokens: Retrieves all currently configured tokens.
  • peblar.add_rfid_token: Adds a new token using a UID and description.
  • peblar.remove_rfid_token: Deletes a specific token by its UID.

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

Copilot AI review requested due to automatic review settings April 26, 2026 13:06
@syphernl syphernl requested a review from frenck as a code owner April 26, 2026 13:06
@home-assistant home-assistant Bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Apr 26, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 26, 2026

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 11d3e4b
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/69f122b1842e5b0008505b00
😎 Deploy Preview https://deploy-preview-45015--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Comment thread source/_integrations/peblar.markdown Outdated
@home-assistant home-assistant Bot marked this pull request as draft April 27, 2026 08:08
@home-assistant
Copy link
Copy Markdown

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@syphernl syphernl marked this pull request as ready for review April 27, 2026 08:21
@home-assistant home-assistant Bot requested a review from c0ffeeca7 April 27, 2026 08:21
c0ffeeca7
c0ffeeca7 previously approved these changes Apr 27, 2026
Copy link
Copy Markdown
Contributor

@c0ffeeca7 c0ffeeca7 left a comment

Choose a reason for hiding this comment

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

Thank you, @syphernl 👍

@c0ffeeca7 c0ffeeca7 added the awaits-parent Awaits the merge of an parent PR label Apr 27, 2026
Comment thread source/_integrations/peblar.markdown Outdated
Comment on lines +236 to +242
### Action: List RFID tokens

The `peblar.list_rfid_tokens` action returns all RFID tokens currently configured in the charger's standalone authorization list.

- **Data attribute**: `config_entry_id`
- **Description**: The Peblar charger to retrieve the RFID token list from.
- **Optional**: No
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Actions now have their dedicated pages. Please create a page for each action in the _actions folder.

Thanks! 👍

../Frenck

                       

Blogging my personal ramblings at frenck.dev

@home-assistant home-assistant Bot marked this pull request as draft April 27, 2026 16:27
Frank Klaassen added 3 commits April 27, 2026 18:35
Documents three new actions added in home-assistant/core#169189:
peblar.list_rfid_tokens, peblar.add_rfid_token, and
peblar.remove_rfid_token for managing the charger's standalone
authorization list.
Tables don't render well on mobile; use list format per style guide.
Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

  • The UI version for using these actions are missing.
  • Examples are missing
  • I've commented on the upstream PR on the name of the remove action.
  • The actions listed on the integration page need to be added using the general include available (see, for example light.md how that works; or check the developer documentation).

@syphernl syphernl marked this pull request as ready for review April 28, 2026 21:12
@home-assistant home-assistant Bot requested review from c0ffeeca7 and frenck April 28, 2026 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaits-parent Awaits the merge of an parent PR has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants