Skip to content
This repository has been archived by the owner on Jan 29, 2023. It is now read-only.

added setUDP function #8

Merged
merged 1 commit into from
Feb 24, 2022
Merged

added setUDP function #8

merged 1 commit into from
Feb 24, 2022

Conversation

argo9
Copy link
Contributor

@argo9 argo9 commented Feb 8, 2022

Added a function that allows setting the UDP socket after the declaration of the NTPClient element.
This allows you to use Ethernet and WiFi within the same code and switch between them at run time.

Added a function that allows setting the UDP socket after the declaration of the NTPClient element.
This allows you to use Ethernet and WiFi within the same code and switch between them at run time.
@khoih-prog
Copy link
Owner

khoih-prog commented Feb 8, 2022

HI @argo9

Thanks for your PR.

Could you provide some MRE to demonstrate the problem this PR will fix.

I'm sorry I currently can't spend some time to study the effect of this PR.

Regards,

@khoih-prog
Copy link
Owner

Hi @argo9

I just have some time to look at your PR, which is very good feature to have. I'll merge, then publish a new release, along with some new examples to use the new enhancement for auto-switching between WiFi and Ethernet UDP to update NTP time.

Thanks for your good idea and looking forward to receiving more PRs from you.

Regards,

@khoih-prog khoih-prog merged commit fff8aa9 into khoih-prog:main Feb 24, 2022
khoih-prog added a commit that referenced this pull request Feb 24, 2022
### Releases v3.7.2

1. Add setUDP() function to enable auto-switching between `WiFi` and `Ethernet` UDP instances. Check [added setUDP function #8](#8)
2. Add new examples [ESP_WiFi_ETH_NTPClient_Advanced](https://github.com/khoih-prog/NTPClient_Generic/tree/main/examples/ESP/ESP_WiFi_ETH_NTPClient_Advanced) and [WT32_WiFi_ETH01_NTPClient_Advanced](https://github.com/khoih-prog/NTPClient_Generic/tree/main/examples/WT32_ETH01/WT32_WiFi_ETH01_NTPClient_Advanced) to use the new setUDP() function for auto-switching between `WiFi` and `Ethernet` UDP to update NTP time.
3. Update `Packages_Patches`
khoih-prog added a commit that referenced this pull request Feb 24, 2022
### Releases v3.7.2

1. Add setUDP() function to enable auto-switching between `WiFi` and `Ethernet` UDP instances. Check [added setUDP function #8](#8)
2. Add new examples [ESP_WiFi_ETH_NTPClient_Advanced](https://github.com/khoih-prog/NTPClient_Generic/tree/main/examples/ESP/ESP_WiFi_ETH_NTPClient_Advanced) and [WT32_WiFi_ETH01_NTPClient_Advanced](https://github.com/khoih-prog/NTPClient_Generic/tree/main/examples/WT32_ETH01/WT32_WiFi_ETH01_NTPClient_Advanced) to use the new setUDP() function for auto-switching between `WiFi` and `Ethernet` UDP to update NTP time.
3. Update `Packages_Patches`
khoih-prog added a commit that referenced this pull request Feb 24, 2022
### Releases v3.7.2

1. Add setUDP() function to enable auto-switching between `WiFi` and `Ethernet` UDP instances. Check [added setUDP function #8](#8)
2. Add new examples [ESP_WiFi_ETH_NTPClient_Advanced](https://github.com/khoih-prog/NTPClient_Generic/tree/main/examples/ESP/ESP_WiFi_ETH_NTPClient_Advanced) and [WT32_WiFi_ETH01_NTPClient_Advanced](https://github.com/khoih-prog/NTPClient_Generic/tree/main/examples/WT32_ETH01/WT32_WiFi_ETH01_NTPClient_Advanced) to use the new setUDP() function for auto-switching between `WiFi` and `Ethernet` UDP to update NTP time.
3. Update `Packages_Patches`
@khoih-prog
Copy link
Owner

Hi @argo9

The new NTPClient_Generic releases v3.7.2 has just been published. Your contribution is noted in Contributions and Thanks

Please check the new examples as well as the debug terminal output when running example ESP_WiFi_ETH_NTPClient_Advanced

Best Regards,


Releases v3.7.2

  1. Add setUDP() function to enable auto-switching between WiFi and Ethernet UDP instances. Check added setUDP function #8
  2. Add new examples ESP_WiFi_ETH_NTPClient_Advanced and WT32_WiFi_ETH01_NTPClient_Advanced to use the new setUDP() function for auto-switching between WiFi and Ethernet UDP to update NTP time.
  3. Update Packages_Patches

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants