Skip to content
This repository has been archived by the owner on Nov 15, 2021. It is now read-only.

Feature events #91

Merged
merged 34 commits into from
Nov 19, 2017
Merged

Feature events #91

merged 34 commits into from
Nov 19, 2017

Conversation

metachris
Copy link
Contributor

EventHub and logging.

Many commits, many changes. Will add some documentation to the README / docs in a later step. A prompt user doesn't really notice much, except a bit more log messages, and the option to use config debug on/off and config sc-events on/off.

For devs that use neo-python as library, this is a great many essential changes. It allows to send all the log messages into logfiles, and to easily react to smart contract events.

metachris and others added 30 commits November 12, 2017 12:33
* origin/development:
  updated docs conf
  updated docs conf
  initial docs
  replaced neo.Network.RPC package with neo-python-rpc pip package
  fixed bootstrap file in protocol.testnet
  finished RPC client implementation
  added documentation to client calls
  adding rpc client
  fix codestyle
  updating docs for Wallet
  bugfix for contrib/privnet-claim-neo-and-gas.py
  Remove white spaces from docstring endings as per @metachris review's request
  initial contrib/privnet-claim-neo-and-gas.py
  Dockerfile/Readme Ubuntu update, added --testnet to prompt args, some minor cleanup
  Add some more docstrings
  Remove void from docstrings, replaced with None where appropriate.
  quick fix CreateKey
  Small cleanup in (User)Wallet.CreateKey functions and a lot of docstrings for `Wallet`
  resolve if the whole file is not found
localhuman and others added 4 commits November 18, 2017 07:59
* Logging: configure through settings, enable logfile

* Update all __log and print statements to new logger system

* move default log level to INFO, and only have certain (often-occuring) log messages in DEBUG

* Fixes

* UserWallet: logger.info -> logger.error + logger.debug

* UserWallet pep8 fix
* origin/development:
  Update all the requirements (#87)
  Added Wallet.GetSyncedBalances
@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 67.732% when pulling 863b408 on feature-events into 1d0684b on development.

@localhuman
Copy link
Collaborator

Just wanted to leave a comment here about the new way to subscribe to SC events:

smart_contract = SmartContract("6537b4bd100e514119e3a7ab49d520d20ef2c2a4")
 
@smart_contract.on_notify
def sc_notify(event):
  print("SmartContract Runtime.Notify event:", event)
  if len(event.event_payload):
    print(event.event_payload[0].decode("utf-8"))

Great work by @metachris !

@localhuman localhuman merged commit daf700d into development Nov 19, 2017
@metachris metachris deleted the feature-events branch November 20, 2017 17:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants