Skip to content
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

Add dialog to insert user fields for board that require them to upload #550

Merged
merged 11 commits into from
Nov 25, 2021

Conversation

silvanocerza
Copy link
Contributor

@silvanocerza silvanocerza commented Oct 14, 2021

This PR adds a dialog for the user to enter custom fields necessary to upload to certain boards that require them.

If the user clicks the upload button on the toolbar and never upload to that board on that addres in the current IDE session the dialog will be shown, if all the fields are filled and the user clicks the Upload button the values will be cached for the current session. Subsequent uploads triggered with the upload button won't ask the user again to enter the fields but will use the cached ones.
If the user wants to change the fields after the first upload she must open the dialog from Sketch > Configure And Upload menu, edit the fields and click the Upload button again.

In any case clicking the Cancel button doesn't save the entered values.

User fields are not saved when closing the IDE.

Example:

upload.mp4

@silvanocerza silvanocerza added the type: enhancement Proposed improvement label Oct 14, 2021
@silvanocerza silvanocerza self-assigned this Oct 14, 2021
@silvanocerza silvanocerza force-pushed the scerza/upload-with-configuration branch from 276bf31 to 24f2ce7 Compare October 15, 2021 15:19
@silvanocerza silvanocerza marked this pull request as ready for review October 15, 2021 15:38
@silvanocerza silvanocerza force-pushed the scerza/upload-with-configuration branch 4 times, most recently from 1e66d57 to febf70a Compare October 20, 2021 13:09
@per1234 per1234 added the topic: code Related to content of the project itself label Oct 20, 2021
@silvanocerza silvanocerza force-pushed the scerza/upload-with-configuration branch from febf70a to d744aa9 Compare November 8, 2021 10:40
@AlbyIanna AlbyIanna force-pushed the scerza/upload-with-configuration branch from b245326 to e5ab9b5 Compare November 24, 2021 14:52
@AlbyIanna AlbyIanna force-pushed the scerza/upload-with-configuration branch from e5ab9b5 to a330d65 Compare November 24, 2021 15:56
Copy link

@ubidefeo ubidefeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functionality tested and confirmed working

@fstasi fstasi merged commit a090dfe into main Nov 25, 2021
@joyel24
Copy link

joyel24 commented Jan 24, 2023

@silvanocerza Hello, thanks for contributing to the project but, can you please fix #1488 as you implemented this features that causes troubles to everyone trying to upload sketch via OTA on their ESP32 devices without password.
Or even add the ability to enter empty field and confirm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants