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

Note to consider private chains on ASSET evaluation #282

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nnnarvaez
Copy link

@nnnarvaez nnnarvaez commented Mar 30, 2019

The current state only evaluates for the 3 main assets for private chains this blocks operations.
check /steembase/chains.py

from chain import know_chains
Iterate ASSETS

I tried to fetch the current prefix from steemd.chain_params

    @property
    def chain_params(self):
        """ Identify the connected network. This call returns a
            dictionary with keys chain_id, prefix, and other chain
            specific settings
        """
        props = self.get_dynamic_global_properties()
        chain = props["current_supply"].split(" ")[1]

        assert chain in known_chains, "The chain you are connecting " + \
                                      "to is not supported"
        return known_chains.get(chain)

Leaving it here to come back to it when time allows, quick and dirty solution would be to just replace your ASSET names there or use the NAI

The current state only evaluates for the 3 main assets for private chains this blocks operations. 
check `/steembase/chains.py` 

```
from chain import know_chains
Iterate ASSETS```

I tried to fetch the current prefix from `steemd.chain_params`

```
    @Property
    def chain_params(self):
        """ Identify the connected network. This call returns a
            dictionary with keys chain_id, prefix, and other chain
            specific settings
        """
        props = self.get_dynamic_global_properties()
        chain = props["current_supply"].split(" ")[1]

        assert chain in known_chains, "The chain you are connecting " + \
                                      "to is not supported"
        return known_chains.get(chain)

```
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.

1 participant