bitshares-ui icon indicating copy to clipboard operation
bitshares-ui copied to clipboard

Beos dialog

Open szechem opened this issue 6 years ago • 8 comments

General

Implementation of a new EOS Bitshares section in Deposit / Withdrawal. It contains a description of the token distribution and functionality of the opening dialog, in which it is possible to create BEOS accounts and transfer tokens between the Bitshares and BEOS networks.

Code Preparation

Please review all your changes one last time before committing

  • [x] Check for unused code
  • [x] No unrelated changes are included
  • [x] None of the changed files are reformatting only
  • [x] Code is self explanatory or documented
  • [x] All written text is properly translated (english language)

Testing

The branch has been tested on the following browsers (desktop and mobile view)

  • [x] Chrome
  • [ ] Opera
  • [x] Firefox
  • [ ] Safari

User interface changes

Delete this section if there weren't any UI changes. Please make sure you tested your changes in all themes

  • [x] Dark
  • [x] Light
  • [x] Midnight

Please provide screenshots/licecap of your changes below

szechem avatar Jul 12 '19 14:07 szechem

beos_a beos_b

szechem avatar Jul 12 '19 14:07 szechem

My own opinion: the "BitShares EOS" text is confusing, better replace all with "BEOS".

abitmore avatar Jul 13 '19 11:07 abitmore

This essentially opens the BTS withdraw functionality and include external custodians to be listed in a similar way like gateways are.

This opens up couple points for discussion:

  1. do we want to include those in the wallet? I feel like yes, since people would have a better UX if they can select to deposit e.g. into binance. Question remains where to get the needed information.
  2. if 1 is yes, which external custodians should we include? opt-in like gateways? community selected?
  3. the legacy Deposit/Withdraw is deprecated and will be removed soon (I just removed the legacy Send last week), adding this now seems not feasible. This needs proper UX and be included in the Deposit/Withdraw modal.

It's a significant strategic change, thus it needs an issue for discussion imo. Any and all new gateways that want to get added and use existing UX need to get approved, and this has new UX on top of it. Please open an issue and we discuss it there how to proceed.

Are you affiliated with BEOS @szechem ? What are your thoughts?

sschiessl-bcp avatar Jul 15 '19 12:07 sschiessl-bcp

szechem is a contractor for BLCA (association for BitShares EOS). These code changes have been tested for a number of months now in a mildly customized version of the reference web wallet hosted by BLCA. IMO, it's a very small feature: it formats a memo when sending bitshares funds. It has clear text that describes risks involved in this operation. I've been requested to create a new issue to discuss inclusion of this PR (or some undesigned idea to replace it, which I think shouldn't delay this PR), so I've made one here: https://github.com/bitshares/bitshares-ui/issues/2950

dnotestein avatar Jul 16 '19 15:07 dnotestein

Please continue discussion in https://github.com/bitshares/bitshares-ui/issues/2950#issuecomment-550231231

sschiessl-bcp avatar Nov 19 '19 09:11 sschiessl-bcp

stale, if not please reopen

sschiessl-bcp avatar Apr 30 '20 22:04 sschiessl-bcp

AFAIK, it's not particularly "stale". Currently the changes are being run on one hosting, because these changes weren't included into the reference wallet. Ideally, they would still be included in the reference wallet.

dnotestein avatar May 01 '20 15:05 dnotestein

Apparently there's conflicts now. I'm not opposed to fixing those, if the changes will be included. But if they are just going to be rejected again, I don't want to waste my time.

dnotestein avatar May 01 '20 15:05 dnotestein