-
Notifications
You must be signed in to change notification settings - Fork 166
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
Root directory is cluttered. Move files to a folder. #102
Comments
While on the one hand that seems a good Idea on the other hand it is difficult if the files are named differently on the computer and the ESP. |
How do you transfer files to the NodeMCU? |
I am using this code in init.lua:
It automatically compiling all ".lua" files and move all files named not "^httpserver*" to "http/". Maybe somebody can make PR from this code. |
@marcelstoer currently I use ESPlorer. I don't want to install python on my (windows) machine right now. I also have a similar script to rename the http files after upload, but not as sophisticated as @ATAMAH s |
@HHHartmann did you mean to tag @marcoskirsch rather than me? |
I've customized my fork of ESPlorer to give a "Save Folder" button specifically so I could work with this project. It also has a dist dir with a prebuilt jar, for those who don't want to download the entire SDK and compile. :3 |
When this project started, all the server code was in a single file. Now it's spread across lots of little files, all in the root directory:
httpserver-b64decode.lua
httpserver-basicauth.lua
httpserver-compile.lua
httpserver-conf.lua
httpserver-connection.lua
httpserver-error.lua
httpserver-header.lua
httpserver-init.lua
httpserver-request.lua
httpserver-static.lua
httpserver-wifi.lua
httpserver.lua
I'm toying with the idea of reorganizing and moving all the server files into a folder. I'd remove the prefix to each file. The folder may be named httpserver/ or simply server/
I'm not about to go and do this, as it can be pretty disruptive to other contributors. So I'm floating the idea here in order to get some feedback.
Thanks!
The text was updated successfully, but these errors were encountered: