-
Notifications
You must be signed in to change notification settings - Fork 244
OTA enhancement. Split port and uploadPort in arduino.json #362
Conversation
We might need differentiate the upload port with the serial port or OTA port. Current there are two issues here:
- We only show one port on the UI but provide two ports. This will confuse the user. We need provide multiple port on the UI yet we don't want to crowd the status bar; people already complain the croweded status bar.
- The change is not compatible with the old schema of arduino.json. Please take a look the file of ./misc/arduinoValidator.json For change on this file should be careful.
@yaohaizh thank you for your feedback! Regading your comments.
- Indeed, the name might be confusing for some users. Maybe an improvement would be to rename it to otaIP or a name that does not look like a serial port at all. Regarding the UI, since this is a specific feature it might be acceptable for most users to just set the ip in a configuration file and do not have a command for this. Just an idea.
- I updated arduino.json and included the field, but it seems to me that i am overlooking something more.
yaohaizh try to do this, but not check in code at last, need to discuss again how to do it
@sernaleon sorry for the slow response, we are working on your pr, thank you for your contribution.
@liydu, from UX perspective, please help to check / design how to arrange 2 ports in the status bar.
@Sneezry and @czgtest please verify it.
lanouettea
commented
Jul 4, 2018
Any update about this? I would really like to be able to upload my sketch using OTA.
hairlesshobo
commented
Feb 2, 2019
Any update on this? I upload all my sketches using a programmer but need to use the serial monitor as well. From the description, it sounds like it wilt enable functionality to keep serial monitor open when uploading without being forced to change the serial port and close the serial monitor. This functionality is what I am used to with my previous arduino development platform (a plug in for visual studio) and would be a huge time-saver for me.
lersi
commented
Apr 22, 2024
hello, I created a community fork at https://github.com/vscode-arduino/vscode-arduino.
You are more than welcome to merge your changes there.
This PR adds a new (optional) property in arduino.json, uploadPort. If defined, the user can specify an IP address or another COM port without closing the serial monitor. If not defined, uploadPort will default to port value, keeping the existing functionality.
Features:
Possible improvements: