Resolves an issue reported via TestFlight. The App shows default values for server port (8080) and tcp port (9090) in the placeholders of the text fields. If the values are not set explicitly, these defaults should be used automatically. This was not done for the server port, which resulted in connection errors in this case.
This PR now changes few things to improve the behaviour and maintainability:
Set the default for server / tcp port via newly introduced helper methods
Use app-wide macros for the default values and show these value as placeholders in the UI
Summary for release notes
Improvement: Use port defaults if not explicitly set
Maintenance: Centralized definition of port defaults
wutschel
commented
1 week ago
Description
Resolves an issue reported via TestFlight. The App shows default values for server port (8080) and tcp port (9090) in the placeholders of the text fields. If the values are not set explicitly, these defaults should be used automatically. This was not done for the server port, which resulted in connection errors in this case.
This PR now changes few things to improve the behaviour and maintainability:
Summary for release notes
Improvement: Use port defaults if not explicitly set Maintenance: Centralized definition of port defaults