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

About the Ethereum blockchain #699

Closed
wants to merge 2 commits into from

Conversation

jorge-campo
Copy link
Contributor

@jorge-campo jorge-campo commented Jul 24, 2023

Hello @Fabiomorais87 – Here is a new article for your review. Thanks!

Note: I can't link the issue for some unknown reason.

@jorge-campo jorge-campo added doc-new Additions to the Status documentation E:GettingStarted Getting started section and subsections labels Jul 24, 2023
@jorge-campo jorge-campo self-assigned this Jul 24, 2023
@Fabiomorais87
Copy link
Contributor

Fabiomorais87 commented Aug 10, 2023

Hi @jorge-campo , here are my review suggestions:

First, I would like to say that I liked the article. This article "looks simple", but in reality, it is a hard one. I say this because you needed to explain different topics in a simple way to the user. The whole time that I was reading your article, I had in mind a regular user that does not understand much about technology (like my dad). And we this in mind I can say that the article achieves its purpose. Still, I have minor suggestions that if it makes sense you could add to the article.

What's Ethereum?

"When we talk about Ethereum, we often refer to the main Ethereum blockchain or mainnet. On mainnet, Ether has a value, and transactions have an actual cost. Besides the mainnet, there are other test Ethereum networks or testnets, where Ether holds no real value."

I understand your point here, but in my opinion, I would take this whole paragraph out. With the regular user in mind, this paragraph would confuse the user more than aggregate more information.

Why do I think like this? The regular user does not know what is a mainnet or a testnet. This information here without more context (which clearly could not be added here) would confuse the regular user. The more advanced user already knows that.

The role of Ethereum in the Status app

In my opinion, this intro could be expanded and linked to upcoming H3s. How about listing the following H3s in this intro? I am saying this because the H2 is the role of Ethereum in the Status app, the intro says that you can benefit from Ethereum in the app, but my following questions are "How?" and "Why should I use this app?". So in this way, in a concise way I would link this H2 to the upcoming H3s and in this way I would answer the "how" and "why" questions. Makes sense what I am saying?

Status crypto wallet

"You can send, receive and store tokens securely, including Ether and ERC-20 tokens,"

I would add a cross-link to "ERC-20 tokens" to the article named as "Undestand about ERC-20 tokens".
(https://github.com/status-im/status-website/issues/380)

Running dApps on Ethereum

I would add a crooss-link to an article named as "Use DApps with the Status Browser". (#122)

General suggestion

The article would benefit greatly if you added an image (header or in the introduction) that describes a blockchain. You may reuse the image that is in the article titled "About Status messages".

Screenshot_9

@jorge-campo jorge-campo force-pushed the 438-about-the-ethereum-blockchain branch from d0175ab to b0d3663 Compare August 11, 2023 12:30
@jorge-campo
Copy link
Contributor Author

Hey @Fabiomorais87 – Thanks for your review.

Here is what I have done:

  • I have expanded the information about mainnet vs testnets. Why not remove this information altogether? Because there are a few screens in the Wallet and Settings section where we mention these two terms. It will be helpful for users to understand this a bit, if only for this reason. But I see your point and have discussed the section a bit further.
  • I added the link to the ERC-20 tokens you suggested but not the one to the 'Use DApps with the Status Browser' article. The project needs to clarify some open questions with the web3 browser first.
  • As per the illustration, I was never happy with that one 😅. I'd prefer Design creates the illustrations and headers for us using the Status brand style and guidelines.

@jorge-campo
Copy link
Contributor Author

Article is ready to merge in the status-website repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-new Additions to the Status documentation E:GettingStarted Getting started section and subsections
Projects
Status: Published/Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants