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.

Sign up for GitHub

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

Jump to bottom

Fix: Use ESP_LOGI instead of ESP_LOGE to log the start of scanning process #1229

Closed
wants to merge 1 commit into from
Closed

Fix: Use ESP_LOGI instead of ESP_LOGE to log the start of scanning process #1229

wants to merge 1 commit into from

Conversation

DominikPalo
Copy link
Contributor

In the spp_client_demo example, the information about the start of scanning process was incorrectly logged as an error.

@CLAassistant
Copy link

CLAassistant commented Nov 5, 2017 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@projectgus
Copy link
Contributor

Thanks @DominikPalo . I've cherry-picked this into our internal review & merge queue.

@projectgus projectgus added the Status: Pending blocked by some other factor label Nov 16, 2017
igrr pushed a commit that referenced this pull request Nov 21, 2017
@DominikPalo @projectgus
Update spp_client_demo.c
ad8ebe5 
Merges #1229
@igrr
Copy link
Member

igrr commented Nov 21, 2017

Thanks for the PR @DominikPalo, it was cherry-picked as ad8ebe5.

@igrr igrr closed this Nov 21, 2017
@igrr igrr removed the Status: Pending blocked by some other factor label Aug 9, 2018
@Elijahg Elijahg mentioned this pull request Sep 25, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@DominikPalo @CLAassistant @projectgus @igrr