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

Add info component on gas prices in the onboarding #28

Closed
javieralaves opened this issue Nov 19, 2019 · 9 comments
Closed

Add info component on gas prices in the onboarding #28

javieralaves opened this issue Nov 19, 2019 · 9 comments

Comments

@javieralaves
Copy link

javieralaves commented Nov 19, 2019

Add a warning info component in the "Review information" section before launching your organization where to inform the organization administrator about gas prices and how to set up a moderate gas price for a smooth setup before clicking on moving onto the final step.

There are two versions for this message depending on whether the user's Ethereum Provider is or isn't MetaMask. Text specifications available here. Link would direct users to the troubleshooting Help Scout documentation.

Figma file available here.

D 1 0 - Review information

@javieralaves javieralaves transferred this issue from AutarkLabs/open-enterprise Nov 19, 2019
@stellarmagnet
Copy link
Collaborator

stellarmagnet commented Nov 19, 2019

The thing I'm worried about with the warning on this screen, is that the user may be confused and try to adjust the gas price there - they won't always click the documentation, I assume some may try before. They won't be able to actually update the gas price until they hit "launch organization". We may want to describe that more so they don't fumble to change the price while they are on this screen.

@javieralaves
Copy link
Author

@stellarmagnet Good point, we could instead suggest the user to activate MetaMask's "Advanced gas controls" so that the gas price can be adjusted right before submitting each transaction. For this to be effective, we should also make clear what the reasonable gas price should be.

@stellarmagnet
Copy link
Collaborator

@jayalaves no, they don't have to enable that and we shouldn't instruct them to turn it on. The way to update the gas without enabling the advanced feature is the hit the "EDIT" link in the Gas Fee section. I was just saying, we need better language so the user knows that the gas price cannot be edited until the MetaMask extension is triggered, which happens after the user hits "Launch Organization"

@stellarmagnet
Copy link
Collaborator

Maybe something like...

Your current gas price is set to 1 Gwei, which means your organization might take about 10 to 15 minutes to set up. You can change this in your MetaMask extension/Ethereum provider as soon as you click the "Launch Organization" button below. Learn more about gas prices and how to adjust them in the documentation.

@sohkai
Copy link
Collaborator

sohkai commented Nov 19, 2019

I don't know about adding a warning here; I'd just increase the default gas price we use based on the API call and place a warning in the signing step with a link to https://ethgas.io/.

@stellarmagnet
Copy link
Collaborator

@sohkai good call - I was on the fence with if we needed this as well, since we are going down the gas API route.

@stellarmagnet
Copy link
Collaborator

@sohkai I'm not sure about that link though, did you mean https://ethgasstation.info/ ? Even with that, the estimate won't be accurate. I think linking to the Aragon help docs would be better (a new page), but we can chat on that more in the #29 ticket

@sohkai
Copy link
Collaborator

sohkai commented Nov 19, 2019

ethgas.io is page I usually give people if they need to understand gas, but yes, in this case, we could provide other links if we wanted to direct users to places to check the current gas market. Having a article in the helpscout docs (see aragon/help) that aggregates those links makes sense.

@javieralaves
Copy link
Author

Thank you everyone for your feedback on this issue – I think we have enough of a common vision as to how to move forward on this particular topic – following @sohkai's recommendation:

I'd just increase the default gas price we use based on the API call and place a warning in the signing step

And we can continue the conversation in #29, as that's where there's still some more inputs we can discuss on potential changes or enhancements to the onboarding process.

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

No branches or pull requests

3 participants