Modifications to led controller example #824
Conversation
MISO and MOSI are referred to as DO and DI on the sd card module now.
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
|
I signed the CLA after being notified. |
| @@ -418,7 +419,7 @@ void setup() { | |||
| } | |||
|
|
|||
| void loop() { | |||
| if (digitalRead(BUTTON_PIN) == LOW) { | |||
| if (digitalRead(BUTTON_PIN) == HIGH) { //check if button is pressed | |||
There was a problem hiding this comment.
In the diagram there's a pull down resistor with the button - guessing you're using a different setup? This will likely change for people depending on what button they're using.
There was a problem hiding this comment.
I followed the diagram with the pull-down resistor and the button, but the recording that is supposed to occur upon button press only worked for me when I modified this line. Otherwise, it would record when the button is not being pressed.
It could be possible that I made a wiring mistake, so feel free to ignore this change.
On newer module versions, MISO and MOSI pins are referred to as DO and DI.
HTTP requests require a content-length header to prevent a 411 error.
The code was originally made to record audio when the button is not pressed, but it was changed to do the opposite.