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 quotes to config backup file name, fixes #2484 #2490

Closed
wants to merge 4 commits into from
Closed

Add quotes to config backup file name, fixes #2484 #2490

wants to merge 4 commits into from

Conversation

Leseratte10
Copy link

This fixes #2484 (incomplete backup file name when friendly name has spaces) by adding quotation marks around the file name.

@arendst
Copy link
Owner

arendst commented Apr 19, 2018

I really appreciate your input.

I go for replacing all spaces in the filename with underscores as I hate spaces so hold on.

@arendst arendst added the enhancement Type - Enhancement that will be worked on label Apr 19, 2018
arendst added a commit that referenced this pull request Apr 19, 2018
5.12.0m
* Fix configuration filename truncation when it contains spaces (#2484,
#2490)
@arendst arendst closed this Apr 19, 2018
arendst added a commit that referenced this pull request Apr 19, 2018
5.12.0m
* Fix configuration filename truncation when it contains spaces (#2484,
#2490)
arendst added a commit that referenced this pull request Apr 19, 2018
5.12.0m
* Fix configuration filename truncation when it contains spaces (#2484,
#2490)
curzon01 pushed a commit to curzon01/Tasmota that referenced this pull request Sep 6, 2018
5.12.0m
* Fix configuration filename truncation when it contains spaces (arendst#2484,
arendst#2490)
curzon01 pushed a commit to curzon01/Tasmota that referenced this pull request Sep 6, 2018
5.12.0m
* Fix configuration filename truncation when it contains spaces (arendst#2484,
arendst#2490)
curzon01 pushed a commit to curzon01/Tasmota that referenced this pull request Sep 6, 2018
5.12.0m
* Fix configuration filename truncation when it contains spaces (arendst#2484,
arendst#2490)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type - Enhancement that will be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Name of 'Backup Configuration' file truncated if Friendly Name contains 'spaces'
2 participants