Skip to content

Conversation

@rcooke-warwick
Copy link
Contributor

The list of supported wifi dongles and modems is out of date and sets an uneralistic expectation for hardware compatibility - its better to remove this until we actually have a proper modem story

Change-type: patch

Supersedes #2193 if we go with this


Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

The list of supported wifi dongles and modems is out of date and sets an uneralistic expectation for hardware compatibility - its better to remove this until we actually have a proper modem story

Change-type: patch
Signed-off-by: Ryan Cooke <ryan@balena.io>
@flowzone-app
Copy link
Contributor

flowzone-app bot commented Jan 8, 2025

Website deployed to CF Pages, 👀 preview link https://363a559f.balenacloud-docs.pages.dev

@flowzone-app flowzone-app bot enabled auto-merge January 8, 2025 10:37
Copy link
Member

@vipulgupta2048 vipulgupta2048 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From my side it's a +1 to merge, trying not to trigger the auto-merge.
@rcooke-warwick can you also comment the following files being pulled from the OS document. We should do a follow up PR to remove this information from the OS docs as well.

  1. Comment line 37 to 42
    # Extract modem text
    ../../tools/extract-markdown.sh "Modems" <meta-balena.md >supported-modems.md
    # Extract Wifi adapters
    ../../tools/extract-markdown.sh "WiFi Adapters" <meta-balena.md >supported-wifi-adapters.md
    # Extract Recommended WiFi USB dongle
    ../../tools/extract-markdown.sh "Recommended WiFi USB dongle" <meta-balena.md >supported-wifi-usb-dongle.md

@rcooke-warwick
Copy link
Contributor Author

thanks @vipulgupta2048 - do you mean I should comment out/delete what you've highlighted ?

@vipulgupta2048
Copy link
Member

@rcooke-warwick Comment out the lines highlighted, we can remove them once the other PR merges in the OS readme and the information no longer exists.

Change-type: patch
Signed-off-by: Ryan Cooke <ryan@balena.io>
@flowzone-app flowzone-app bot merged commit cddaa24 into master Jan 14, 2025
53 checks passed
@flowzone-app flowzone-app bot deleted the ryan/dongle branch January 14, 2025 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants