This repository has been archived by the owner on Mar 28, 2023. It is now read-only.
bulkupdatecurrency fails if the user has crypto listings #1632
Comments
cpacia
added a commit
that referenced
this issue
Jun 12, 2019
The API would fail if the user had a cryptocurrency listing set since we only allow one accepted currency for cryptolistings. This commit changes it to not apply the bulk update to cryptolistings and adds a test. Closes #1632
cpacia
added a commit
that referenced
this issue
Jun 13, 2019
The API would fail if the user had a cryptocurrency listing set since we only allow one accepted currency for cryptolistings. This commit changes it to not apply the bulk update to cryptolistings and adds a test. Closes #1632
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
If a user has one or more cryptocurrency listings, and tries to set more than one currency using bulkupdatecurrency the call will fail with a
a cryptocurrency listing must only have one accepted currencyerror.The call works if only one currency is sent.
I think the bulkupdatecurrency call should ignore any cryptocurrency listings, it's unlikely anyone would want to update those all at once to be for a different target currency.
The text was updated successfully, but these errors were encountered: