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

Allow to specify different upstream endpoint #3994

Closed
adambabik opened this issue Apr 25, 2018 · 14 comments
Closed

Allow to specify different upstream endpoint #3994

adambabik opened this issue Apr 25, 2018 · 14 comments
Labels
Milestone

Comments

@adambabik
Copy link
Contributor

User Story

As a user, I'd like to use Status with upstream node but I'd like to specify the upstream address by myself.

Description

Type: Feature

Summary: It's important that even if one decides to use upstream node, there is a choice. Currently, the upstream networks use infura and it's not possible to change it.

Solution

Summary:

  • allow specifying a custom upstream address when selecting the upstream network
@oskarth
Copy link
Contributor

oskarth commented Apr 26, 2018

Do we need some designs for this? Can we re-use components that we use for other custom data entry? Would probably be easiest

cc @chadyj @denis-sharypin

@status-open-bounty
Copy link

status-open-bounty commented May 1, 2018

Balance: 0.000000 ETH
Tokens: SNT: 3000.00
Contract address: 0x55c5c9daf2ef4f9914c5e9c995b91f755c07c524
Network: Mainnet
Paid to: ConnorChristie
Visit https://openbounty.status.im to learn more.

@debragail
Copy link
Contributor

is there a design for this?

@ConnorChristie
Copy link

@debragail @oskarth @chadyj @denis-sharypin
How does an added option to the network settings look? Possibly bringing up a modal for user input when selected: (I also think the networks should be categorized)

image

@adambabik
Copy link
Contributor Author

I like the categories! It looks nice.

One thing I would change is Testnet category. testnet term is interchangeable with Ropsten (from geth -h output):

  --testnet                                  Ropsten network: pre-configured proof-of-work test network
  --rinkeby                                  Rinkeby network: pre-configured proof-of-authority test network

To avoid confusion, maybe we can call this category Public test networks or something similar that is descriptive and avoids using a single term testnet.

What's gonna happen when the user taps on Custom upstream RPC? I believe there should appear a modal window to enter the URL and, similarly to Connected label, it should display the currently entered upstream RPC address.

And the final case, what if one would like to have multiple custom upstream RPC addresses? It may be a valid case.

@ConnorChristie
Copy link

ConnorChristie commented May 9, 2018

Instead of the mockup that only includes the ability to add one custom network, maybe there could be a plus button in the top right to add a custom network and those would show under the custom section?

As a side note, at first glance when switching to a testnet, I found the two different (local and upstream) options for each network confusing. Would it be intuitive if they were merged into one option and upon selection a modal is shown that allows them to either select local or upstream?

@adambabik
Copy link
Contributor Author

That sounds good! Then the flow would be (1) tap + to display the modal to enter a custom network address, (2) after tapping OK the modal disappears, (3) the new custom network is displayed in the list. I guess we would also need to provide a way to delete custom networks and maybe edit option.

@ConnorChristie
Copy link

The deletion could be the infamous swipe row to show delete button. It isn't the most intuitive but it will most likely be devs or power users adding and removing custom upstreams so I am sure they'd be able to figure it out since many other apps use the swipe to delete functionality

@debragail
Copy link
Contributor

I think swipe row for deletion is a good ux in this case,

@oskarth oskarth added the p2p label May 10, 2018
@oskarth
Copy link
Contributor

oskarth commented May 10, 2018

@ConnorChristie good idea!

Let's remove design blocked, better get something working here for release candidate and we can add designs later.

@oskarth
Copy link
Contributor

oskarth commented May 10, 2018

cc @mandrigin mobile app

@adambabik do we need to do anything on P2P side for this?

@adambabik
Copy link
Contributor Author

I don't think so. Currently, when I switch to an upstream network, I am presented with a view with a JSON config and the upstream URL. This should work the same for custom upstream URLs so no changes on status-go side should be required.

@oskarth
Copy link
Contributor

oskarth commented May 10, 2018

@ConnorChristie do you want to give this a go? It's one of our blockers for beta release :)

@mandrigin Since we aren't pushing for improvements to LES (instead awaiting ULC) this would be very useful from a decentralization POV.

@ConnorChristie
Copy link

@oskarth Sure, I can work on this!

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

No branches or pull requests

8 participants