UDP port number is 50222 not 5022

masjoelin commented 5 months ago

Feedback

UDP port number is 50222 not 5022 for weatherflow udp port. See doc at https://weatherflow.github.io/Tempest/api/udp/v171/)

URL

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

Version

2024.5.1

Additional information

No response

home-assistant[bot] commented 5 months ago

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

Code owner commands

Code owners of `weatherflow` 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 weatherflow` 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.

jeeftor commented 5 months ago

This is a documentation issue right?Sent from my iPhoneOn May 5, 2024, at 7:45 AM, home-assistant[bot] @.***> wrote: Hey there @natekspencer, @jeeftor, mind taking a look at this feedback as it has been labeled with an integration (weatherflow) you are listed as a code owner for? Thanks!

Code owner commands Code owners of weatherflow 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 weatherflow 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.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

masjoelin commented 5 months ago

Yeah, doc list 5022 instead of 50222. Noticed when I setup by udp relay today

Den sön 5 maj 2024 15:48Jeef @.***> skrev:

This is a documentation issue right?Sent from my iPhoneOn May 5, 2024, at 7:45 AM, home-assistant[bot] @.***> wrote: Hey there @natekspencer, @jeeftor, mind taking a look at this feedback as it has been labeled with an integration (weatherflow) you are listed as a code owner for? Thanks!

Code owner commands Code owners of weatherflow 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 weatherflow 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.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/home-assistant.io/issues/32592#issuecomment-2094816653, or unsubscribe https://github.com/notifications/unsubscribe-auth/A33DFXSKKH6N2BHAOFT3TGLZAY2CJAVCNFSM6AAAAABHHXV5TSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJUHAYTMNRVGM . You are receiving this because you authored the thread.Message ID: @.***>

natekspencer commented 5 months ago

I'm seeing 50222 in both places: Screenshot_20240505_134806_Chrome Screenshot_20240505_134839_Chrome

jeeftor commented 5 months ago

I’m confused :)Sent from my iPhoneOn May 5, 2024, at 1:50 PM, Nathan Spencer @.***> wrote: I'm seeing 50222 in both places: Screenshot_20240505_134806_Chrome.jpg (view on web) Screenshot_20240505_134839_Chrome.jpg (view on web)

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

masjoelin commented 5 months ago

Close this issue. Will check tomorrow where I thought I saw 5022.

Den sön 5 maj 2024 21:53Jeef @.***> skrev:

I’m confused :)Sent from my iPhoneOn May 5, 2024, at 1:50 PM, Nathan Spencer @.***> wrote: I'm seeing 50222 in both places: Screenshot_20240505_134806_Chrome.jpg (view on web) Screenshot_20240505_134839_Chrome.jpg (view on web)

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/home-assistant.io/issues/32592#issuecomment-2094928533, or unsubscribe https://github.com/notifications/unsubscribe-auth/A33DFXTI5TNACJQZRTR244TZA2E5PAVCNFSM6AAAAABHHXV5TSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJUHEZDQNJTGM . You are receiving this because you authored the thread.Message ID: @.***>

natekspencer commented 5 months ago

@home-assistant close

jeeftor commented 5 months ago

Also feel free to make a pr if you want. Always good to get more contributions and doc fixes are easy :)

home-assistant / home-assistant.io