Browse Source

remove 3rd button from ligthning wallets

master
ncoelho 5 years ago
committed by Overtorment
parent
commit
a11d4aa453
  1. 2
      loc/en.js
  2. 11
      screen/wallets/transactions.js

2
loc/en.js

@ -27,7 +27,7 @@ module.exports = {
empty_txs1_lightning:
'Lightning wallet should be used for your daily transactions. Fees are unfairly cheap and speed is blazing fast.',
empty_txs2_lightning: '\nTo start using it tap on "manage funds" and topup your balance.',
tap_here_to_buy: 'buy Bitcoin',
tap_here_to_buy: 'Buy Bitcoin',
},
reorder: {
title: 'Reorder Wallets',

11
screen/wallets/transactions.js

@ -218,7 +218,16 @@ export default class WalletTransactions extends Component {
return (
<View style={{ flex: 1 }}>
<View style={{ flexDirection: 'row', margin: 16, justifyContent: 'space-evenly' }}>
{this.state.wallet.getTransactions().length > 0 && this.renderSellFiat()}
{/*
So the idea here, due to Apple banning native Lapp marketplace, is:
On Android everythins works as it worked before. Single "Marketplace" button that leads to LappBrowser that
opens /marketplace/ url of offchain wallet type, and /marketplace-btc/ for onchain is not, but on the details.
On iOS its more complicated - we have one button that opens same page _externally_ (in Safari), and second
button that opens actual LappBrowser but with _blank_ page. This is important to not trigger Apple.
Blank page is also the way Trust Wallet does it with Dapp Browser.
For ONCHAIN wallet type no LappBrowser button should be displayed, its Lightning-network specific.
*/}
{this.state.wallet.getTransactions().length > 0 && this.state.wallet.type !== LightningCustodianWallet.type && this.renderSellFiat()}
{this.state.wallet.type === LightningCustodianWallet.type && this.renderMarketplaceButton()}
{this.state.wallet.type === LightningCustodianWallet.type && Platform.OS === 'ios' && this.renderLappBrowserButton()}
</View>

Loading…
Cancel
Save