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

[Feature] Network module: Network connected but no internet format #2372

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Absobel
Copy link

@Absobel Absobel commented Aug 1, 2023

Attempt to resolve #1487.

It seem to work with wifi but I couldn't test it with ethernet because I lack an ethernet port on my laptop.

The network module will try pinging the list of servers in the array "ping-servers" (the module will keep its default behaviour if it is empty or not initialised) every "interval" seconds. I also added the format for WiFi without internet "format-wifi-no-internet" and the format for ethernet without internet "format-ethernet-no-internet" if the pings fail

@Absobel Absobel changed the title Connected no internet [Feature] Network module: Network connected but no internet format Aug 18, 2023
@Absobel
Copy link
Author

Absobel commented Mar 6, 2024

Hey, I saw that this pull request never had any comments or anything. Would it be possible to review my code and tell me if I need to modify it, or if it is an eldritch horror of an abomination, or anything in between ?

I guess my pull request just went unnoticed x) (tbh i forgot it too)

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

Successfully merging this pull request may close these issues.

[Feature] Network module: Network connected but no internet format
1 participant