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

Reference code fixed. (IDFGH-9066) #10469

Closed
wants to merge 1 commit into from
Closed

Reference code fixed. (IDFGH-9066) #10469

wants to merge 1 commit into from

Conversation

mushsoooup
Copy link
Contributor

@mushsoooup mushsoooup commented Dec 31, 2022

Fixed a small code bug in reference document.

PS: I have no idea why this goes wrong when the english version is correct.

@CLAassistant
Copy link

CLAassistant commented Dec 31, 2022

CLA assistant check
All committers have signed the CLA.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Dec 31, 2022
@github-actions github-actions bot changed the title Reference code fixed. Reference code fixed. (IDFGH-9066) Dec 31, 2022
@Alvin1Zhang
Copy link
Collaborator

Thanks for your contribution.

@@ -22,7 +22,7 @@ HTTP Server 组件提供了在 ESP32 上运行轻量级 Web 服务器的功能
esp_err_t get_handler(httpd_req_t *req)
{
/* 发送回简单的响应数据包 */
const char[] resp = "URI GET Response";
const char resp[] = "URI GET Response";
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would you please fix Line58 as well, it has the same issue.

@mahavirj
Copy link
Member

mahavirj commented Jan 4, 2023

PS: I have no idea why this goes wrong when the english version is correct.

It was fixed in #3940, but seems that we missed out on propagating the fix to Chinese version of the docs.

@hmalpani
Copy link
Contributor

hmalpani commented Jan 9, 2023

sha=27fc115b98127094790067c82abb0a3237b9f97a

@hmalpani hmalpani added PR-Sync-Merge Pull request sync as merge commit and removed PR-Sync-Merge Pull request sync as merge commit labels Jan 9, 2023
@espressif-bot espressif-bot added Status: In Progress Work is in progress Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Opened Issue is new Status: In Progress Work is in progress Resolution: NA Issue resolution is unavailable labels Jan 9, 2023
espressif-bot pushed a commit that referenced this pull request Jan 13, 2023
@mahavirj
Copy link
Member

Merged with ed1b2ba

@mahavirj mahavirj closed this Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-Sync-Merge Pull request sync as merge commit Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants