ideaHex / FHTbot

Firmware for Makerfaire robot
Apache License 2.0
10 stars 7 forks source link

Final pre-makerfaire Turtle Page #50

Closed Rodinga closed 8 years ago

Rodinga commented 8 years ago

This one should fix the stop button issue I mentioned, the abort issue, changes the background, and adds a 6000 character limit to any commands sent to the bot. (Doing that should cause an unresponsive script error in a web-browser anyway.)

TildenG commented 8 years ago

Its showing something strange about the path, some of the file's your are uploading are in WebControls/public_html/ ??? , we could run into problems with the path being too long again.

And the stop command should look like this dxhttp.open("GET", "/data,AS,0,", true); not dxhttp.open("GET", "/data,AS", true);. It could cause intermittent crashes.

You've changed the heartbeat timer back to 100ms, there's no need to send this much data, the heartbeat has a 1000ms timeout so I recommend a setting of 300 - 490ms, this will help with browsers we haven't tested yet.

Please remove the @ in any reference to FHTbot. eg line 767 TurtleMode.html.

TildenG commented 8 years ago

It still shows the TurtleMode.html in "WebControls/public_html/TurtleMode.html".

It shows you deleted TurtleMode.html from data/T/TurtleMode.html ????.

Rodinga commented 8 years ago

The HTML version of turtle mode was removed to force use of the .gzip version, it's not a small file. The webcontrols folder itself has been deleted from the master branch and is confined to a development branch. That should fix the /??? path issue as well.