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.

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

Settings: Connect to specific node from mobile wallet #98

Closed
kaminskyelina opened this issue Apr 10, 2019 · 5 comments
Closed

Settings: Connect to specific node from mobile wallet #98

kaminskyelina opened this issue Apr 10, 2019 · 5 comments
Assignees

Comments

@kaminskyelina
Copy link
Contributor

No description provided.

@kaminskyelina kaminskyelina changed the title Connect to specific node from mobile wallet Settings: Connect to specific node from mobile wallet Apr 11, 2019
@alexandrashelenkova
Copy link

screens are here
zpl.io/25OMeK8
zpl.io/V0L0Dow

@alexandrashelenkova
Copy link

Setting screen updated, added one more screen here zpl.io/2pvneqQ

@sinaizd
Copy link

sinaizd commented Apr 19, 2019

  1. Why IP and port are located in two different lines? @alexandrashelenkova
    image
  2. I think we need to decide which option to display node or peer. @alexandrashelenkova
    image
  3. User сan not save settings if delete port, IP or delete both. If you remove ip:port, in this case, the status of the wallet should be an error: "Wallet is not connected to node: {old node address} {net name}".@DenisDemyanko
  4. Must be IP instead of Address. @DenisDemyanko
    image
  5. Make the words brighter. @DenisDemyanko
    image

@alexandrashelenkova
Copy link

  1. @sasha-abramovich asked to make two fields in commets in zeplin
  2. Let's keep "node", I will update screens in zeplin later
  3. It gonna be default alert same as incompatible node, I will add screen later too, but I think that one might be done without actual design for now?
    4,5. Screen for testnet should be same as for mainnet in design (just different background color). So it should be ip:port in screen title, and text in fields should be white

@sinaizd
Copy link

sinaizd commented Apr 23, 2019

checked

@sinaizd sinaizd closed this as completed Apr 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants