-
Notifications
You must be signed in to change notification settings - Fork 7
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
Enhancements #6
Open
BikeMikeAU
wants to merge
64
commits into
eppye-bots:master
Choose a base branch
from
BikeMikeAU:enhancements
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Enhancements #6
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
Replaced media/admin with a new version from Django, this fixes a lot of upgrade issues. 1.8 will be minimum version supported now. Also created a few new svg icons in the images folder to replace the old gifs.
Found during testing
syntax error indent
fix ini key for daily logging to match botsinit.py print to console all severe errors
First ran 2to3, then made some manual fixes. There is some code to allow backwards compatibility with python 2 but unlikely it will fully work. Need to decide if Py2 is really wanted. If not, some code can be removed or improved.
Submit_line needed to be replaced from later version of Django, I added cancel button mack in. confirm.py fix missing %}
paramiko dependency error mention version > 2.0 RSA keyfile and passphrase can be configured in channel without user script needed
1. match variable names to the settings - less confusing reading the code :-) 2. maxconnectiontries applies to both in and out channels. 3. fix the try-except-else-finally (i had to read python docs again... break in finally clause was no good) 4. new python 3 exception context adds confusion to the trace when there is a connection error; fix that to just re-raise the original (raise exc from exc)
Also check for exceptions and default to no indenting
cleanup was already added but setting was not in bots.ini Also fix help link to open new window/tab
although environment is not used by CherryPy now, it is useful to keep this setting. Can be referred to in mapping scripts etc to take different action in development (such as not sending emails)
was hard-coded as 2 seconds, this did not work well for me on windows.
Config startup delay to allow for automated windows patching and reboots. Config to allow duplicate jobs on queue Config to kill process after maxruntime timer expires
Can configure named groups of routes, and run those routes by passing -g<group name> to engine
Use to filter attachments based on filename
Major version should increment due to backwards incompatible changes https://semver.org/
Django updated jquery, so need to also update jquery-ui and timepicker to get the widget working again
There are many online discussions and articles on removal of terms with racist connotations in software. Looks like this small change is the only one in Bots code.
Clearer help text on screen. Allow deletion by making rightcode blank in csv
base.html and submit_line.html templates are refreshed from Django so show a lot if diffs. remove images not needed.
was for python 2.6
was for python 2.6
py3 does not need to import cPickle, cDecimal, cElementtre etc simplejson/json StringIO/io ConfigParser/configparser persistfilters remove from settings, now default behavior in Django
got errors when processing edifact files containing UNA because this was not skipped. CONTRL sender and recipient were not set.
I'm going to continue using this "Enhancements" branch for any bug fixes too. In hindsight it was too complex managing multiple branches for different changes. |
I got this error: Traceback (most recent call last): File "C:\D\PythonEnv\Bots330_Py39\lib\site-packages\bots\transform.py", line 81, in _translate_one_file for inn_splitup in edifile.nextmessage(): #for each message in parsed edifile (one message might get translation multiple times via 'alt' File "C:\D\PythonEnv\Bots330_Py39\lib\site-packages\bots\inmessage.py", line 363, in nextmessage ta_info.update(eachmessage.queries[-1]) AttributeError: 'list' object has no attribute 'queries' Don't fully understand the code but deduced that this was the fix needed :-)
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.
These are most of my enhancements uplifted from 3.2.