Skip to content
This repository was archived by the owner on Apr 3, 2019. It is now read-only.

Conversation

AdamDanielKing
Copy link
Contributor

…e and SecurityTokenRegistrar to the Polymath class. Remove SecurityTokenRegistrar's dependency on a SecurityToken.

…e and SecurityTokenRegistrar to the Polymath class. Remove SecurityTokenRegistrar's dependency on a SecurityToken.
@davekay100
Copy link
Contributor

it looks good , any idea why the Travis and the netlify are failing? can i merge anyways?

@ChainSafeSystems
Copy link
Contributor

@priom what are your thoughts?

@priom
Copy link
Contributor

priom commented Feb 13, 2018

Can u guys work on passing the test? I see there are some changes on the wrappers and tests.

Copy link
Contributor

@davekay100 davekay100 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, the updates for access to securiyt tokens and templates has been updated, as well as the fixes to the npm module, travis, and netlify . accepting

@davekay100 davekay100 merged commit def6a27 into master Feb 13, 2018
@AdamDanielKing AdamDanielKing deleted the publish-fix branch February 13, 2018 21:35
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.

6 participants