Import/ Export cleanup #70
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 9 commits
February 26, 2023 10:22
config.ini changed depending upon lines, hence replaced with a variable
config.ini changed depending upon lines, hence replaced with a variable
config.ini changed depending upon lines, hence replaced with a variable
…e location rather than relative when run interactively
…annot see anything surrounding it to suggest that the script order should include this before the exit, hence its removal
Changed backup_filename to include the full path as this was used in all bar one place Created an import_location variable to enforce consistency through the script Changed the check for import directory to remove existing as a bland and simple means to clear all files including . prefixed files Changed emonhub.conf code to check for emonhub config directory first and copy if exists or leave a warning message if not
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Cleanup of readme.md to include "cd backup" to ensure the user is in the correct directory first.
Changes to emoncms-export.sh, emoncms-import.sh and usb-import.sh to correct location of module.json, making config.cfg locations consistent as some output differs from the real location and change the script location to always show the full path even when triggered by ./emoncms-export.sh (approach copied from usb-import.sh)
I've tested the scripts without a config.cfg.
The other scripts provide effectively identical output.
And when config.cfg exists:
Again the other scripts provide an effectively identical output.