You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Jan 29, 2023. It is now read-only.
Copy file name to clipboardExpand all lines: CONTRIBUTING.md
+25-4Lines changed: 25 additions & 4 deletions
Original file line number
Diff line number
Diff line change
@@ -15,7 +15,7 @@ If you don't find anything, please [open a new issue](https://github.com/khoih-p
15
15
Please ensure to specify the following:
16
16
17
17
* Arduino IDE version (e.g. 1.8.19) or Platform.io version
18
-
* Board Core Version (e.g. ESP32 core v2.0.4)
18
+
* Board Core Version (e.g. ESP32 core v2.0.5)
19
19
* Contextual information (e.g. what you were trying to achieve)
20
20
* Simplest possible steps to reproduce
21
21
* Anything that might be relevant in your opinion, such as:
@@ -28,26 +28,47 @@ Please ensure to specify the following:
28
28
```
29
29
Arduino IDE version: 1.8.19
30
30
WT32_ETH01 board
31
-
ESP32 core v2.0.4
31
+
ESP32 core v2.0.5
32
32
OS: Ubuntu 20.04 LTS
33
-
Linux xy-Inspiron-3593 5.15.0-46-generic #49~20.04.1-Ubuntu SMP Thu Aug 4 19:15:44 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
33
+
Linux xy-Inspiron-3593 5.15.0-53-generic #59~20.04.1-Ubuntu SMP Thu Oct 20 15:10:22 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
34
34
35
35
Context:
36
-
I encountered a crash while trying to connect to network.
36
+
I encountered a crash while using this library
37
37
38
38
Steps to reproduce:
39
39
1. ...
40
40
2. ...
41
41
3. ...
42
42
4. ...
43
43
```
44
+
45
+
---
46
+
44
47
### Sending Feature Requests
45
48
46
49
Feel free to post feature requests. It's helpful if you can explain exactly why the feature would be useful.
47
50
48
51
There are usually some outstanding feature requests in the [existing issues list](https://github.com/khoih-prog/WebServer_WT32_ETH01/issues?q=is%3Aopen+is%3Aissue+label%3Aenhancement), feel free to add comments to them.
49
52
53
+
---
54
+
50
55
### Sending Pull Requests
51
56
52
57
Pull Requests with changes and fixes are also welcome!
53
58
59
+
Please use the `astyle` to reformat the updated library code as follows (demo for Ubuntu Linux)
60
+
61
+
1. Change directory to the library GitHub
62
+
63
+
```
64
+
xy@xy-Inspiron-3593:~$ cd Arduino/xy/WebServer_WT32_ETH01_GitHub/
<ahref="https://www.buymeacoffee.com/khoihprog6"title="Donate to my libraries using BuyMeACoffee"><imgsrc="https://cdn.buymeacoffee.com/buttons/v2/default-yellow.png"alt="Donate to my libraries using BuyMeACoffee"style="height: 50px!important;width: 181px!important;" ></a>
9
10
<ahref="https://www.buymeacoffee.com/khoihprog6"title="Donate to my libraries using BuyMeACoffee"><imgsrc="https://img.shields.io/badge/buy%20me%20a%20coffee-donate-orange.svg?logo=buy-me-a-coffee&logoColor=FFDD00"style="height: 20px!important;width: 200px!important;" ></a>
@@ -113,22 +116,22 @@ Please have a look at [HOWTO Fix `Multiple Definitions` Linker Error](#howto-fix
113
116
114
117
#### Important notes
115
118
116
-
ESP32 Core v2.0.0+ introduces new enum breaking almost all `WT32_ETH01` codes written for core v1.0.6-.
119
+
ESP32 Core `v2.0.0+` introduces new enum breaking almost all `WT32_ETH01` codes written for core `v1.0.6-`.
117
120
118
121
It's really strange to define a breaking enum `arduino_event_id_t` in [**WiFiGeneric.h**#L36-L78](https://github.com/espressif/arduino-esp32/blob/master/libraries/WiFi/src/WiFiGeneric.h#L36-L78), compared to the old `system_event_id_t`, now placed in [**esp_event_legacy.h**#L29-L63](https://github.com/espressif/arduino-esp32/blob/master/tools/sdk/esp32/include/esp_event/include/esp_event_legacy.h#L29-L63)
119
122
120
123
It's better to preserve the old enum order and just adding new items **to do no harm to pre-2.0.0 codes**
121
124
122
-
- Releases v1.2.0- to be used for ESP32 core v1.0.6- only
123
-
- Releases v1.3.0 can be used for either ESP32 core v2.0.0+ or v1.0.6-. **Default is using core v2.0.0+**
125
+
- Releases `v1.2.0-` to be used for ESP32 core `v1.0.6-` only
126
+
- Releases `v1.3.0` can be used for either ESP32 core `v2.0.0+` or `v1.0.6-`. **Default is using core `v2.0.0+`**
124
127
125
128
To use with core v1.0.6-, just define in your sketch
126
129
127
-
```
130
+
```cpp
128
131
#defineUSING_CORE_ESP32_CORE_V200_PLUS false
129
132
```
130
133
131
-
- Releases v1.4.0+ can be used for either ESP32 core v2.0.0+ or v1.0.6-. **Autodetect core**
134
+
- Releases `v1.4.0+` can be used for either ESP32 core `v2.0.0+` or `v1.0.6-`. **Autodetect core**
132
135
133
136
134
137
#### Features
@@ -164,8 +167,8 @@ This [**WebServer_WT32_ETH01** library](https://github.com/khoih-prog/WebServer_
164
167
## Prerequisites
165
168
166
169
1.[`Arduino IDE 1.8.19+` for Arduino](https://github.com/arduino/Arduino). [](https://github.com/arduino/Arduino/releases/latest)
167
-
2.[`ESP32 Core 2.0.4+`](https://github.com/espressif/arduino-esp32) for ESP32-based WT32_ETH01 boards using release v1.3.0+. [](https://github.com/espressif/arduino-esp32/releases/latest/)
168
-
3.[`ESP32 Core 1.0.6-`](https://github.com/espressif/arduino-esp32) for ESP32-based WT32_ETH01 boards using release v1.2.0-
170
+
2.[`ESP32 Core 2.0.5+`](https://github.com/espressif/arduino-esp32) for ESP32-based WT32_ETH01 boards using release `v1.3.0+`. [](https://github.com/espressif/arduino-esp32/releases/latest/)
171
+
3.[`ESP32 Core 1.0.6-`](https://github.com/espressif/arduino-esp32) for ESP32-based WT32_ETH01 boards using release `v1.2.0-`
169
172
170
173
---
171
174
---
@@ -213,14 +216,14 @@ The current library implementation, using `xyz-Impl.h` instead of standard `xyz.
213
216
214
217
You can include this `.hpp` file
215
218
216
-
```
219
+
```cpp
217
220
// Can be included as many times as necessary, without `Multiple Definitions` Linker Error
in many files. But be sure to use the following `.h` file **in just 1 `.h`, `.cpp` or `.ino` file**, which must **not be included in any other file**, to avoid `Multiple Definitions` Linker Error
222
225
223
-
```
226
+
```cpp
224
227
// To be included only in main(), .ino with setup() to avoid `Multiple Definitions` Linker Error
The following are debug terminal output and screen shot when running example [AdvancedWebServer](examples/AdvancedWebServer) on ESP32_DEV with ETH_PHY_LAN8720 (WT32_ETH01)
<ahref="https://www.buymeacoffee.com/khoihprog6"title="Donate to my libraries using BuyMeACoffee"><imgsrc="https://cdn.buymeacoffee.com/buttons/v2/default-yellow.png"alt="Donate to my libraries using BuyMeACoffee"style="height: 50px!important;width: 181px!important;" ></a>
10
+
<ahref="https://www.buymeacoffee.com/khoihprog6"title="Donate to my libraries using BuyMeACoffee"><imgsrc="https://img.shields.io/badge/buy%20me%20a%20coffee-donate-orange.svg?logo=buy-me-a-coffee&logoColor=FFDD00"style="height: 20px!important;width: 200px!important;" ></a>
0 commit comments