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

docs: improve readability, consistency, and formatting #334

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

sergeypanin1994
Copy link

  • Fixed typos and improved clarity across multiple documentation files
  • Standardized formatting for better readability
  • Updated outdated links and descriptions
  • Improved UI components and deployment guides for Celo Composer
  • Enhanced step descriptions for a smoother setup and deployment experience
  • Updated Twitter URL to x.com to reflect rebranding
  • Standardized contact links formatting and improved wording for better guidance

Updated the Twitter URL from https://twitter.com to https://x.com to reflect the platform's rebranding.
- Renamed "Check Docs" to "Documentation" for clarity
- Improved wording for better readability and user guidance
- Ensured consistent punctuation across descriptions
- Fixed typos and improved readability
- Standardized formatting for better clarity
- Enhanced step descriptions for a smoother deployment experience
- Fixed typos and improved readability
- Standardized formatting for clarity
- Enhanced step descriptions for a smoother setup experience

docs: improved deployment guide for Celo Composer

- Fixed typos and improved readability
- Standardized formatting for better clarity
- Enhanced step descriptions for a smoother deployment experience
- Fixed typos and improved consistency
- Standardized formatting for better readability
- Updated outdated links and enhanced descriptions
Copy link

netlify bot commented Feb 18, 2025

👷 Deploy request for celo-composer pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit e2d2053


- Step 4: Choose to Use a Template: You will be asked if you want to use a template. Select `Yes` or `No`.
## **Built With**
Celo Composer is built on the Celo ecosystem and supports multiple frameworks and libraries:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Celo Composer is built on Celo and supports multiple frameworks and libraries:"

3. **Select a template** (Minipay, Valora, etc.)
4. **Provide project owner details**
5. **Wait for project creation to complete**
6. **Follow the on-screen instructions to start your project**

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keep the original instructions.

1. Change `packages/hardhat/env.template` to `packages/hardhat/env` and add your `PRIVATE_KEY` into the `.env` file.
2. Make sure your wallet is funded when deploying to testnet or mainnet. You can get test tokens for deploying it on Alfajores from the [Celo Faucet](https://faucet.celo.org/alfajores).
3. Run the following commands from the `packages/hardhat` folder to deploy your smart contract to the Celo Testnet Alfajores:
Quick steps:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's call this "Quickstart"

3. Run the following commands from the `packages/hardhat` folder to deploy your smart contract to the Celo Testnet Alfajores:
Quick steps:
1. Rename `packages/hardhat/.env.template` to `packages/hardhat/.env` and add your `PRIVATE_KEY`.
2. Ensure your wallet has test funds from the [Celo Faucet](https://faucet.celo.org/alfajores).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sentence might not be clear to someone who is just starting in Web3

  1. Ensure your wallet is funded before deploying a contract. You can get test tokens from the Celo Faucet.

Copy link
Contributor

@GigaHierz GigaHierz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks Great. Thank you so much for your contribution. Please implement the comments and this can be merged.


2. Open the newly renamed .env file and add your WalletConnect Cloud Project ID from [WalletConnect Cloud](https://cloud.walletconnect.com/)
## **Deploy Your Dapp Locally**
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add

Find the detailed instructions on how to run your frontend in the [`react-dapp` README.md](./packages/react-app/README.md).

### Quickstart


Once you've done that, you're all set to start your project!
1. Rename `.env.template` to `.env` in `packages/react-app/`.
2. Add your WalletConnect Project ID from [WalletConnect Cloud](https://cloud.walletconnect.com/).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Add your WalletConnect Project ID from WalletConnect Cloud in the .env file.

## **Supported Templates**
### **Minipay**
- Pre-configured for building a mini-payment dApp on Celo
📖 [Minipay Docs](https://docs.celo.org/developer/build-on-minipay/overview)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's actually "MiniPay"

## **Support**
Join the Celo community on Discord:
📌 [Celo Discord](https://chat.celo.org)
💬 [Repo Support Channel](https://discord.com/channels/600834479145353243/941003424298856448)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While we are at it, the link to the dedicated repo should be removed

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

Successfully merging this pull request may close these issues.

2 participants