Skip to content
This repository has been archived by the owner on Oct 27, 2020. It is now read-only.

Account & Faucet UI #25

Merged
merged 29 commits into from Mar 5, 2019
Merged

Account & Faucet UI #25

merged 29 commits into from Mar 5, 2019

Conversation

kremalicious
Copy link
Contributor

@kremalicious kremalicious commented Feb 26, 2019

Port over account popup from Pleuston and make the status indicator a shared component. Popup shows on hover wherever we show the indicator in the UI.

Put faucet functionality onto a new route /faucet and link to it from account popup.

ToDo

  • faucet function
  • get and show network name
  • add some new popup library to handle popup appearing in all places
  • get and show balances
  • investigate and fix window overflow for popup in header
  • investigate and fix missing account (states.account is empty in popup for whatever reason)
  • create new faucet UI as its own page with proper feedback
  • verify if faucet action actually works
  • output new account Ether balance in success message (probably pass original balance through in this.getTokens() function)

Current state

screen shot 2019-03-01 at 15 45 02

screen shot 2019-02-28 at 13 39 45

screen shot 2019-02-28 at 13 40 08

@jernejpregelj
Copy link
Contributor

Make it rain function will not be there in a future. We have to use faucet functions for that.

@kremalicious
Copy link
Contributor Author

ah yes, that's what I meant :-) Added that in df97dd2 by using what's being worked on in #24

@kremalicious kremalicious marked this pull request as ready for review February 27, 2019 12:43
@kremalicious kremalicious changed the title Account popup Account & Faucet UI Feb 28, 2019
@jernejpregelj
Copy link
Contributor

Added balances and network name to context. Both need to wait for next version of squid-js

@jernejpregelj jernejpregelj self-requested a review March 5, 2019 14:35
@jernejpregelj jernejpregelj merged commit 214c535 into master Mar 5, 2019
@kremalicious kremalicious deleted the feature/account-popup branch March 5, 2019 15:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants