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 file length check before upload ota #1086

Merged
merged 8 commits into from
Sep 27, 2022
Merged

Add file length check before upload ota #1086

merged 8 commits into from
Sep 27, 2022

Conversation

haverland
Copy link
Collaborator

@haverland haverland commented Sep 26, 2022

  • add check if filename not longer than 30 characters (Rolling - Test New Update Mechanism #1068 )
  • fix handling after error dialog
  • added update scenario to release notes
  • added "from 11.3.1" files to release artifacts

@@ -168,6 +168,8 @@ <h2>Reboot</h2>
alert("No file selected!");
} else if (filePath.length == 0) {
alert("File path on server is not set!");
} else if (filePath.length > 30) {
alert("Filename is to long! Use a shorter name.");
Copy link
Collaborator

Choose a reason for hiding this comment

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

@haverland Please provide the number, else people have to do try&error until it fits the check

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

added

@jomjol jomjol merged commit 0eb70b6 into jomjol:rolling Sep 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants