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

[not an issue] 2 delivery carriers but 3 sensors in HA ? #7

Closed
matfouc opened this issue Feb 24, 2022 · 2 comments · Fixed by #6 or #9
Closed

[not an issue] 2 delivery carriers but 3 sensors in HA ? #7

matfouc opened this issue Feb 24, 2022 · 2 comments · Fixed by #6 or #9
Assignees
Labels
documentation Improvements or additions to documentation enhancement New feature or request question Further information is requested

Comments

@matfouc
Copy link
Contributor

matfouc commented Feb 24, 2022

Hi,

Thanks for your work ! I've submitted a separate PR to add a french translation just in case it could be useful for some others :).
Anyway besides that, i have a question, i see 3 sensors in HA despite there are 2 carriers so far (Postnord and Citymail).
Since the integration is still very recent and documentation is clear but short, i'm wondering the purpose of the 3rd sensor.

  • Is it correct that it simply aggregates both carriers and indicates the next scheduled delivery, independantly of which carrier it is ?

Cheers,
Matthieu

@DSorlov DSorlov self-assigned this Feb 24, 2022
@DSorlov DSorlov added documentation Improvements or additions to documentation enhancement New feature or request question Further information is requested labels Feb 24, 2022
@DSorlov
Copy link
Owner

DSorlov commented Feb 24, 2022

Hi! Will merge it into dev and it will be released very soon! Thanks for the translation!

Regarding the third sensor; it is just as you say a combined sensor with the next delivery regardless of operator (it is actually prepared to handle even more operators if needed)

@matfouc
Copy link
Contributor Author

matfouc commented Feb 24, 2022

All clear, thanks !

@DSorlov DSorlov linked a pull request Feb 26, 2022 that will close this issue
DSorlov added a commit that referenced this issue Feb 26, 2022
Clarifies details as outlined in question in issue #7
DSorlov added a commit that referenced this issue Feb 26, 2022
Documentation update #7
@DSorlov DSorlov mentioned this issue Feb 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants