Skip to content
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

Tx Remarks #123

Closed
wants to merge 8 commits into from
Closed

Tx Remarks #123

wants to merge 8 commits into from

Conversation

uvmetal
Copy link
Collaborator

@uvmetal uvmetal commented Jun 19, 2018

Important Notes:

  1. Some tests were fudged to get code into play so as not to hold up community PRs; will address in test sprint asap.

  2. State tree for network switcher needs a looking at in another PR. This is open for grabs for anyone wanting to contribute or I will resolve soon. This works as-is.

  3. Some of the neon-js code has been removed in favor of a local Neolink api file (NeoscanAPI.js) so as to ease integration and control api needs without need to interrupt or wait on neon-js. Once Neolink neoscan api needs freeze we will look at porting back to neon-js. This is another contribution opportunity.

Details Follow:
Supports Transaction Remarks

Transaction interface now supports remark attribute usage.
Supports Transaction Fees

Added optional parameter to sendAsset.
Private/Custom Network Support

Custom URL support has been added to allow connectivity with any host:port/api.
Promisified Neoscan Support API

Developed Custom Neolink interface to Neoscan and reworked DX.
Updated Network Switcher to Support Neoscan API

Neon-js / NeonDB is still present, but mostly, if not completely, inactive.
This was chosen as a result of time constraints around defusing API documentation difficulties.
Various Usability Features

Transaction Summary Info
Transaction Timestamps on Tx Cards
Debug Helpers

Deep Object Inspection & Logging
Fixed Private Key Storage

Promisified FQDN URL Validation for Custom Networks and Runtime Assurance

Added String Utilities

Rewrote tests to work with Neoscan

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.8%) to 36.623% when pulling 379a4bf on phetter:tx_remarks into 2e9e850 on CityOfZion:development.

@uvmetal uvmetal closed this Jun 19, 2018
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.

None yet

3 participants