Skip to content
This repository has been archived by the owner on May 13, 2022. It is now read-only.

[META] Feedback from demo 05/05/20 and 05/06/20 and 05/12/20 #158

Closed
18 tasks done
fredo opened this issue May 5, 2020 · 4 comments
Closed
18 tasks done

[META] Feedback from demo 05/05/20 and 05/06/20 and 05/12/20 #158

fredo opened this issue May 5, 2020 · 4 comments

Comments

@fredo
Copy link
Contributor

fredo commented May 5, 2020

Here are the most important feedback from the demo with kamal:

Paul's and Kamal's feedback for macOC

Open questions from users:

  • should we already join DAI token network? (Ask every user what she thinks and then we decide)
  • what happens when the user clicks the "back"-button on the browser
@fredo
Copy link
Contributor Author

fredo commented May 6, 2020

It might be possible that some of the errors happened due to clicking twice on a button. How could we mitigate such a problem?
Keep in mind that the backend is supposed to be completely stateless.
Maybe the user clicked twice due to slow reaction time of the UI?

@Dominik1999 Dominik1999 changed the title Feedback from demo 05/05/20 Feedback from demo 05/05/20 and 05/06/20 May 6, 2020
@Dominik1999 Dominik1999 changed the title Feedback from demo 05/05/20 and 05/06/20 [META] Feedback from demo 05/05/20 and 05/06/20 May 6, 2020
@konradkonrad
Copy link
Contributor

konradkonrad commented May 7, 2020

Some follow ups from #109 are inlcuded in the first post

@Dominik1999
Copy link
Contributor

Dominik1999 commented May 12, 2020

Bugs found in the demo of the 12th

Already fixed in f9405a8

  • Wizard fast transaction (1 gwei transaction in Talel's run took forever)

UX Improve explanations

follow up issues #213 and #214

  • add flow at the first page again
  • 3 icons but 6 steps
  • "provide Infura ID" - text is confusing
  • Talel did not understand difference SWAP RDN and SWAP DAI (write in the title step 5/6?)

Mini UX Edits

follow up issue #208

  • Text and spinner are too close
  • Different colours for the amounts and message while swapping in the spinner
  • Always write Raiden in upper case
  • Every Text bigger than 12px (see first page)
  • Move step information to the title
  • view the passphrase when you choose it -> see it in plain text
  • download wallet main purpose, main action was quite hidden

Cost estimates

Add follow up issue

  • cost estimate before swapping (should be faster now?)

Failing TX

follow up issues #209 and #210

  • don't show transaction hash it is too long & confusing (maybe hide it?)
  • user needs feedback what to do with a transaction that is spinning for a long time (link to etherscan)
  • Integrate a link for tx_hash to etherscan
  • What is the user waiting for during a transaction -> more explanation for the user

Handle unexpected user flow

follow up issues #211 and #212

  • we need a process for users shutting down the browser or clicking the back-button
  • handle accidental shutdown

Apple stuff

follow up issue #204

  • need to allow access (make it an executable)
  • code sign with Apple

internal for us

  • next time make binaries 1h before the demo

Open Questions

  • should we allow importing a funded keystore json file (Dominik thinks No)
  • user doesn't know how many tokens are needed -> just offer 1 amount? Don't let the user choose and just use 1 amount?

@Dominik1999 Dominik1999 changed the title [META] Feedback from demo 05/05/20 and 05/06/20 [META] Feedback from demo 05/05/20 and 05/06/20 and 05/12/20 May 13, 2020
@Dominik1999
Copy link
Contributor

Feedback Boris Run:

  • clicking the Wizard does not work
  • user needs to make it an executable
  • Boris needed to run as admin
  • We need to create binaries on an old Linux

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants