status-im / nim-status

MIT License
9 stars 6 forks source link

fix: in getWalletAccount check `chat = FALSE` instead of `wallet = 0` #288

Closed michaelsbradleyjr closed 3 years ago

michaelsbradleyjr commented 3 years ago

We're already checking chat = 0 in getWalletAccounts (plural), and we should do the same in getWalletAccount (singular).

This allows getWalletAccount to retrieve the address for the default created account, a.k.a. Status account. That address should be usable with e.g. the /sendtransaction command.

Also, in SQL statements (and some comments) switch from 0/1 to FALSE/TRUE for boolean columns.

Closes #289.


See also: https://github.com/status-im/nim-status/pull/268#discussion_r692363992.