Skip to content

Added HTTPMethods example#406

Merged
mathieucarbou merged 1 commit intomainfrom
example
Mar 14, 2026
Merged

Added HTTPMethods example#406
mathieucarbou merged 1 commit intomainfrom
example

Conversation

@mathieucarbou
Copy link
Copy Markdown
Member

@mathieucarbou mathieucarbou commented Mar 14, 2026

Adds an HTTPMethods example to show how to use asyncws in projects conflicting with arduino core enum (including HTTP_Method.h)

This example will also help verify further refactoring

Copilot AI review requested due to automatic review settings March 14, 2026 11:27
@mathieucarbou mathieucarbou self-assigned this Mar 14, 2026
@mathieucarbou
Copy link
Copy Markdown
Member Author

@willmmiles FYI

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new PlatformIO example demonstrating HTTP method handling in ESPAsyncWebServer, including compatibility with Arduino HTTP_Method definitions.

Changes:

  • Added examples/HTTPMethods/HTTPMethods.ino demonstrating GET/POST and ANY method routing.
  • Updated platformio.ini to list the new Headers and HTTPMethods examples and reordered one existing example entry.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
platformio.ini Adds commented src_dir entries for new examples and reorders WebDAVMethods entry.
examples/HTTPMethods/HTTPMethods.ino New example showing HTTP method flags and compatibility with HTTP_Method.h.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread examples/HTTPMethods/HTTPMethods.ino Outdated
@mathieucarbou mathieucarbou merged commit 5dbf4f7 into main Mar 14, 2026
33 checks passed
@mathieucarbou mathieucarbou deleted the example branch March 14, 2026 13:00
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.

2 participants