balance-io / old-balance-ios

Balance in Swift
GNU General Public License v3.0
49 stars 16 forks source link

🦊 Make supported wallets clearer #98

Open ricburton opened 5 years ago

ricburton commented 5 years ago

People can’t add Binance or Coinface

wolffan commented 5 years ago

I don't think you can add wallets from exchanges...

On Wed, Mar 27, 2019 at 1:35 AM Richard Burton notifications@github.com wrote:

People can’t add Binance or Coinface

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/balance-io/balance-ios/issues/98, or mute the thread https://github.com/notifications/unsubscribe-auth/ABCp8HACEDQAk7UmECjHPHtXgRAyRpTbks5varzSgaJpZM4cMt4t .

-- Raimon Lapuente Twitter: @wolffan CV: Website http://raimonlapuente.com "No... try not! Do... or do not! There is no try... " -- Yoda

einsteinx2 commented 5 years ago

Depends on the exchange. Some use a single wallet per user that doesn't change (for example Wyre), but I think that's only for deposits and presumably they get moved out to a pool, so you're right it probably won't work with any exchange.

einsteinx2 commented 5 years ago

Maybe we should just make it clear only personal wallets are supported

einsteinx2 commented 5 years ago

@ricburton how would you like this done UI-wise?

ricburton commented 5 years ago

I'll take this one.