Skip to content

Cleaning, Code Restructure, Bug Fixes#6

Closed
well-it-wasnt-me wants to merge 13 commits intothomas-pythonas:masterfrom
zacharytomlinson:cleaning-and-restructure
Closed

Cleaning, Code Restructure, Bug Fixes#6
well-it-wasnt-me wants to merge 13 commits intothomas-pythonas:masterfrom
zacharytomlinson:cleaning-and-restructure

Conversation

@well-it-wasnt-me
Copy link

ATTENTION !

At moment I do not have a machine to test what i did. (plan to have one again in next days).

So, before merging, test this code !

List of Changes

  • Created a folder "dictionaries" where to store all machine response code
  • Created a config class in order to handle the config files
  • Create a template config.yml file
  • Created a class for Error handling
  • Fixed issue about a naming confusion (we had aft_get_last_transaction as var name and function name...so every time instead of calling the function python would call the var)
  • Readme Update
  • Creation of requirements.txt (might be good to define which packet version to use in order to avoid problem in future with eventual lib break changes)
  • Created an example file

dedalgr and others added 13 commits May 20, 2021 17:07
Its working on IGT, EGT, Amatic, Novomatic, Casino Technology (Tested on all)
Don't return False. At RAM CLEAR 0 is False and code is broken.
Do not cache. If you lose SAS it will return the cache. 
Check crc method
…t, config management, readme update, added requirements, small fixes
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