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

Add AltMarkets.io Exchange support to Markets #307

Merged
merged 1 commit into from Nov 20, 2019

Conversation

@TheHolyRoger
Copy link
Contributor

TheHolyRoger commented Nov 11, 2019

This PR adds AltMarkets.io exchange support to markets in Iquidus Explorer.

Visibly working here:
https://explorer.theholyroger.com/markets/altmarkets

TheHolyRoger referenced this pull request in TheHolyRogerCoin/RogerXplorer Nov 11, 2019
@TheHolyRoger TheHolyRoger force-pushed the TheHolyRogerCoin:altmarkets branch from a75bf45 to 2dff260 Nov 11, 2019
@cryptozeny

This comment has been minimized.

Copy link

cryptozeny commented Nov 11, 2019

i will test it in near future

@TheHolyRoger TheHolyRoger force-pushed the TheHolyRogerCoin:altmarkets branch from 2dff260 to 8415436 Nov 11, 2019
@TheHolyRoger

This comment has been minimized.

Copy link
Contributor Author

TheHolyRoger commented Nov 11, 2019

Sorry for another force push but wanted to keep commit clean. Final tweaks to add BTC/exchange volume and 24h change %

Edit: Ah bloody git! That's it now, final force push below. All seems to be working for me.

@TheHolyRoger TheHolyRoger force-pushed the TheHolyRogerCoin:altmarkets branch from 8415436 to ef8af68 Nov 11, 2019
@TheHolyRoger TheHolyRoger changed the title Add Altmarkets Exchange Add AltMarkets.io Exchange support to Markets Nov 11, 2019
@cryptozeny cryptozeny mentioned this pull request Nov 20, 2019
@cryptozeny

This comment has been minimized.

Copy link

cryptozeny commented Nov 20, 2019

  "markets": {
    "coin": "SUGAR",
    "exchange": "BTC",
    "enabled": ["altmarkets"],
    "cryptopia_id": "1658",
    "ccex_key" : "Get-Your-Own-Key",
    "default": "altmarkets"
  },
@cryptozeny

This comment has been minimized.

Copy link

cryptozeny commented Nov 20, 2019

its working

$ npm start # term-1

> explorer@1.6.1 start /home/ak/git/SUGAR/EXPLORER/explorer
> node --stack-size=10000 ./bin/cluster

Starting cluster with pid: 16569
no richlist entry found, creating now..
no stats entry found, creating now..
no altmarkets entry found, creating now..
no stats entry found, creating now..
no stats entry found, creating now..
no altmarkets entry found, creating now..
no altmarkets entry found, creating now..
initial richlist entry created for Sugarchain
initial stats entry created for Sugarchain
initial stats entry created for Sugarchain
initial stats entry created for Sugarchain
initial markets entry created for altmarkets
initial markets entry created for altmarkets
initial markets entry created for altmarkets
$ node scripts/sync.js market
script launched with pid: 16844
altmarkets market data updated successfully.

image

Copy link

cryptozeny left a comment

ef8af68 Tested ACK

@iquidus iquidus merged commit 3e6925a into iquidus:master Nov 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.