Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Web ui #30

Merged
merged 9 commits into from
Nov 9, 2018
Merged

Web ui #30

merged 9 commits into from
Nov 9, 2018

Conversation

hyhydra0
Copy link
Contributor

@hyhydra0 hyhydra0 commented Nov 1, 2018

This adds a web a web interface to the application.

itswisdomagain
itswisdomagain suggested changes Nov 3, 2018
View reviewed changes
web/server.go Show resolved Hide resolved
web/server.go Show resolved Hide resolved
web/server.go Show resolved Hide resolved
@itswisdomagain
Copy link
Contributor

Some issues I noticed with the send functionality:

  1. Send amount says unit is in DCR but I'm not sure it is. I had a balance of 200 DCR, sent 100 "DCR" (twice) and my balance is 199.999492 DCR.

  2. The transaction hash displayed after sending is scrambled
    screenshot 2018-11-03 at 10 20 06 am

  3. Refreshing the page after a send transaction re-submits the previous request (even though it was clearly successful). Feels like bad UX. After a user sends a particular amount to a particular address successfully, if he opts to refresh the page, it's likely he wants to make another transaction not to repeat the previous transaction.

@hyhydra0 hyhydra0 changed the title Web ui [WIP] Web ui Nov 5, 2018
@hyhydra0 hyhydra0 changed the title [WIP] Web ui Web ui Nov 5, 2018
@hyhydra0 hyhydra0 merged commit 5c441d6 into raedahgroup:master Nov 9, 2018
@hyhydra0 hyhydra0 deleted the web-ui branch November 9, 2018 15:13
@itswisdomagain itswisdomagain mentioned this pull request Nov 13, 2018
Closed
itswisdomagain pushed a commit to itswisdomagain/godcr that referenced this pull request Dec 29, 2018
@hyhydra0
Merge pull request raedahgroup#30 from codemaestro64/web-ui
11886cf 
Web ui
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@itswisdomagain itswisdomagain itswisdomagain requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hyhydra0 @itswisdomagain