-
Notifications
You must be signed in to change notification settings - Fork 104
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
Improvement: IPv4, IPv6 and host name can be changed after calling Find Kodi once #1069
Conversation
@kambala-decapitator, I am making this draft as it would be good, if you can test this. I don't want to mess this up right before the release. |
6602acb
to
8cfe336
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems to work fine!
you may also want to fix typos in the first commit message
- Fill server name, address, http port and tcp port only once - Let server data follow selection of ipv4, ipv6 or host name
Introduce helpers to fill select and show the server details for the selected segment.
Embed the whole processing in a condition. Remove the fallback logic as this is not required anymore.
- Increase width for address - Decrease width and font size for ports - Do not scale font size for ports and MAC address - Use thin light gray border
8cfe336
to
404a4b8
Compare
Thanks, fixed this as well. |
Reworked, squashed and rebased to master. |
Description
Suggested by @kambala-decapitator.
"Find Kodi" now only needs to be called once. This is will execute the service discovery for all supported address types. Afterwards the user can select the address type without running "Find Kodi" again.
Important: MAC address can only be resolved using iPv4. The code has been changed to use only IPv4 for this, even if the user selects IPv6 or host name as address type. If IPv4 address cannot be found, the MAC address cannot be resolved.
Summary for release notes
Improvement: IPv4, IPv6 and host name can be selected after calling Find Kodi once