Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Since PR #115108 is IMAP documentation up-to-date? #32732

Closed
kds69 opened this issue May 13, 2024 · 4 comments
Closed

Since PR #115108 is IMAP documentation up-to-date? #32732

kds69 opened this issue May 13, 2024 · 4 comments
Assignees

Comments

@kds69
Copy link

kds69 commented May 13, 2024

Feedback

Documentation still refers to text and headers whereas this PR #115108 is aiming at removing text and headers from data event for the sake of size.
Do I get this right and below example taken from "Using events" section of the IMAP documentation is still pulling event text and headers data?
(https://www.home-assistant.io/integrations/imap)

template:
  - trigger:
      - platform: event
        event_type: "imap_content"
        id: "custom_event"
    sensor:
      - name: imap_content
        state: "{{ trigger.event.data['subject'] }}"
        attributes:
          Entry: "{{ trigger.event.data['entry_id'] }}"
          UID: "{{ trigger.event.data['uid'] }}"
          Message: "{{ trigger.event.data['text'] }}"
          ...
          To: "{{ trigger.event.data['headers'].get('Delivered-To', ['n/a'])[0] }}"
          Return-Path: "{{ trigger.event.data['headers'].get('Return-Path',['n/a'])[0] }}"
          Received-first: "{{ trigger.event.data['headers'].get('Received',['n/a'])[0] }}"
          Received-last: "{{ trigger.event.data['headers'].get('Received',['n/a'])[-1] }}"

URL

https://www.home-assistant.io/integrations/imap/

Version

2024.5.3

Additional information

No response

@home-assistant
Copy link

Hey there @jbouwh, mind taking a look at this feedback as it has been labeled with an integration (imap) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of imap can trigger bot actions by commenting:

  • @home-assistant close Closes the feedback.
  • @home-assistant rename Awesome new title Renames the feedback.
  • @home-assistant reopen Reopen the feedback.
  • @home-assistant unassign imap Removes the current integration label and assignees on the feedback, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information) to the feedback.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information) on the feedback.

@jbouwh
Copy link
Contributor

jbouwh commented May 14, 2024

The example still applies, but could be improved to use the new services

@jbouwh
Copy link
Contributor

jbouwh commented May 14, 2024

The example for the last PR is here:
https://www.home-assistant.io/integrations/imap/#example---post-processing
It is an optional approach. The selection of a config entry ID works okay in the developer services or in a blue print, but in yaml this is a bit fuzzy. Should be okay in the UI though.

@jbouwh
Copy link
Contributor

jbouwh commented May 29, 2024

Closing as the IMAP documentations seems okay

@jbouwh jbouwh closed this as not planned Won't fix, can't repro, duplicate, stale May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants