Skip to content

Doc: WiFi library description: remove specific mention of wifi encryption. #6671

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

Merged
merged 1 commit into from
May 27, 2022

Conversation

Rotzbua
Copy link
Contributor

@Rotzbua Rotzbua commented Apr 30, 2022

Description of Change

Update documentation:
Remove specific mention of wifi encryption as availability depends on compilation parameters.
Otherwise it maybe confuses users if a specific encryption is not available.

Tests scenarios

None.

Related links

No explicit

... as availability depends on compilation parameters.
@github-actions
Copy link
Contributor

github-actions bot commented Apr 30, 2022

Unit Test Results

0 files  0 suites   0s ⏱️
0 tests 0 ✔️ 0 💤 0

Results for commit 15da9a8.

♻️ This comment has been updated with latest results.

@VojtechBartoska VojtechBartoska added the Type: Documentation Issue pertains to Documentation of Arduino ESP32 label May 2, 2022
Copy link
Collaborator

@mrengineer7777 mrengineer7777 left a comment

Choose a reason for hiding this comment

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

LGTM

@me-no-dev me-no-dev requested a review from pedrominatel May 4, 2022 11:32
Copy link
Member

@pedrominatel pedrominatel left a comment

Choose a reason for hiding this comment

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

LGTM

@me-no-dev me-no-dev merged commit 46a026a into espressif:master May 27, 2022
@Rotzbua Rotzbua deleted the patch-2 branch May 27, 2022 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation Issue pertains to Documentation of Arduino ESP32
Projects
Development

Successfully merging this pull request may close these issues.

5 participants