Skip to content

- add RUDEX.PZM, icon and markets#3244

Merged
sschiessl-bcp merged 1 commit intobitshares:developfrom
blckchnd:bitshares-dev
May 3, 2020
Merged

- add RUDEX.PZM, icon and markets#3244
sschiessl-bcp merged 1 commit intobitshares:developfrom
blckchnd:bitshares-dev

Conversation

@siralexpu
Copy link
Copy Markdown
Contributor

  • add RUDEX.PZM, icon and markets
  • add custom field for public key PZM (for withdrawals)
  • change endpoint RuDEX gateApi

@sschiessl-bcp
Copy link
Copy Markdown
Contributor

Could you elaborate please on the need for the public key field plz for withdrawals?

The old endpoint for gateway is still live?

@siralexpu
Copy link
Copy Markdown
Contributor Author

Could you elaborate please on the need for the public key field plz for withdrawals?

The old endpoint for gateway is still live?

Yes - all endpoints are currently supported and operational ..

The public key is needed for output. This is a feature of the blockchain.

The public key is NOT ALWAYS known and can be obtained from the PRIZM blockchain - these are features.
While the account is not activated in the blockchain, information about its public key cannot be obtained, because not available - in this case - the user needs to enter it on his own - the user can find it in his wallet. After activating the account, the public key becomes available for receipt and auto-completion in the corresponding field in DEX UI.

@sschiessl-bcp
Copy link
Copy Markdown
Contributor

Your PR includes a change to the generic WithdrawModal. Would you be able to do do integration tests for all gateways with release candidate?

@siralexpu
Copy link
Copy Markdown
Contributor Author

Your PR includes a change to the generic WithdrawModal. Would you be able to do do integration tests for all gateways with release candidate?

yes, I can check

@sschiessl-bcp
Copy link
Copy Markdown
Contributor

Please rebase onto latest develop

- add custom field for public key PZM (for withdrawals)
- change endpoint RuDEX gateApi
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.

2 participants