RE: SteemAX Update: v0.2 Release
Let me go through the list of Updates you have stated in your Contribution -
Practically the entire code base has been refactored into neatly organized classes instead of module definitions. The code is a thousand times cleaner, easier to read and runs faster. Hundreds of lines of redundant code was removed and made to be more pythonic. - Seems like a Refactoring
A logging and error messaging system as implemented throughout all classes giving a plethora of new information about all SteemAX transactions and invites. This creates a log file named "error.log" and when logging is set at the "INFO" level all transaction activities are logged. - Part of Refactoring
A new MySQL database was created to keep track of all authorization transactions. Each time SteemAX processes its transaction history, a record is made of the transaction including all relevant actions taken. This is used to prevent redundant transactions and keep track of the last transaction that was processed so that new transactions can be identified. - Cannot call it a feature
The authentication process was made more robust, and a number of bugs were removed. Steemax can now be run on my home laptop and perform almost all of its primary functions. Now that's efficiency! - Utopian Rules says "Bug Fixes for contributor’s Own Projects will not be accepted, unless the Bugs were caused by third party dependencies."
Doc strings have been created and comments have been embellished so that they are more informative to those reading the source code. This will be an ongoing process so that as much confusion as possible is dispelled from the inner workings of SteemAX. - Documentation does not mean the actual development
Two new modules were created to handle SteemAX transactions (axtrans.py) and messages printed to screen as well as to the logging file (axmsg.py). - As you said these are the two new modules, agreed.
All transactions are now handled using the steempy cli wallet and the installation instructions have been updated to provide details for configuring the steempy with SteemAX.- Dont call it as a feature though, part of refactoring.
The new command process has been added to the command-line interface. Typing this command will cause SteemAX to run axtrans.py and process all new transactions containing memo IDs. - Wasn't it connected to the axtrans.py.
Now if you go through the "https://review.utopian.io/result/3/2322321", I accept that for this question "How would you rate the knowledge and expertise necessary to fix the bug / implement the added feature(s)?", it should be Average and not Low, that is my mistake.
One more thing, if you are open to take feedback, its better to link the Commit Ids along side the Updates Stated in the contribution. In that way it will be easier for the moderators to evaluate it.