Fix Typo - #69
Conversation
WalkthroughThree documentation edits under learn/getting-started: in README.md a typographical fix removed an extra space before the period after "Keplr" (changed "Keplr ." to "Keplr."); in download-a-mobile-wallet.md two sentences were merged to state availability on the App Store and Google Play and "24 set word keys" was replaced with "24-word seed phrase" (hyphenated); in connect-wallet-to-explorer-app.md an erroneous duplicated trailing link fragment was removed. No functional code, links, control flow, or public API declarations were changed. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
learn/getting-started/README.md (2)
33-35: Fix EVM address prefix ("Ox" ➜ "0x")Ethereum addresses use the 0x prefix; using “Ox” would mislead users.
-$request Ox12345abcde… +$request 0x12345abcde…
95-99: Broken link: "Launch a node/full node/validator" — replace targetThe target is "(broken-reference)". In learn/getting-started/README.md (lines 95–99) replace it with:
../../validate-the-network/run-a-validator-full-node/step-by-step-guide/becoming-a-validator.md
| ### Get a wallet in Testnet | ||
|
|
||
| Testnet Oro supports wallet connections with EVM and Cosmos based wallets like MetaMask and Keplr . To set up a wallet, make sure you have the MetaMask or Keplr wallet extension downloaded in your web browser. MetaMask supports extension downloads for the following web browsers: Chrome, Firefox, Brave, Edge and Opera. Kelpr supports wallets in Chrome, Firefox and Edge. | ||
| Testnet Oro supports wallet connections with EVM and Cosmos based wallets like MetaMask and Keplr. To set up a wallet, make sure you have the MetaMask or Keplr wallet extension downloaded in your web browser. MetaMask supports extension downloads for the following web browsers: Chrome, Firefox, Brave, Edge and Opera. Kelpr supports wallets in Chrome, Firefox and Edge. |
There was a problem hiding this comment.
Fix remaining typo ("Kelpr" ➜ "Keplr"); optional style polish on this sentence
"Kelpr" is misspelled. Also consider minor style tweaks (hyphenation, Oxford comma, “installed” vs “downloaded”) for clarity.
-Testnet Oro supports wallet connections with EVM and Cosmos based wallets like MetaMask and Keplr. To set up a wallet, make sure you have the MetaMask or Keplr wallet extension downloaded in your web browser. MetaMask supports extension downloads for the following web browsers: Chrome, Firefox, Brave, Edge and Opera. Kelpr supports wallets in Chrome, Firefox and Edge.
+Testnet Oro supports wallet connections with EVM- and Cosmos-based wallets like MetaMask and Keplr. To set up a wallet, make sure you have the MetaMask or Keplr wallet extension installed in your web browser. MetaMask supports extension downloads for the following web browsers: Chrome, Firefox, Brave, Edge, and Opera. Keplr supports wallets in Chrome, Firefox, and Edge.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| Testnet Oro supports wallet connections with EVM and Cosmos based wallets like MetaMask and Keplr. To set up a wallet, make sure you have the MetaMask or Keplr wallet extension downloaded in your web browser. MetaMask supports extension downloads for the following web browsers: Chrome, Firefox, Brave, Edge and Opera. Kelpr supports wallets in Chrome, Firefox and Edge. | |
| Testnet Oro supports wallet connections with EVM- and Cosmos-based wallets like MetaMask and Keplr. To set up a wallet, make sure you have the MetaMask or Keplr wallet extension installed in your web browser. MetaMask supports extension downloads for the following web browsers: Chrome, Firefox, Brave, Edge, and Opera. Keplr supports wallets in Chrome, Firefox, and Edge. |
🤖 Prompt for AI Agents
In learn/getting-started/README.md around line 17, fix the typo "Kelpr" to
"Keplr" and revise the sentence for clarity and style: replace "downloaded in
your web browser" with "installed as a browser extension," add an Oxford comma
in the browser list, use consistent hyphenation ("EVM- and Cosmos-based
wallets"), and ensure MetaMask/Keplr names are spelled correctly and
capitalized.
Repeating the same sentence twice within a single paragraph sounds a bit odd, so it should be revised by adding a comma and the conjunction “and.” This change makes the wording flow much better and eliminates the awkwardness. As for the phrase “manage their 24 set word keys,” it is uncommon and somewhat confusing. In the crypto ecosystem it is normally referred to as a **24‑word seed phrase** or a **24‑word recovery phrase**.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
learn/getting-started/download-a-mobile-wallet.md (2)
10-10: Tighten sentence, fix “log in,” and normalize hyphenation.Minor grammar/style nits to improve readability and consistency.
-Mobile wallets are currently in deployment mode for Testnet Oro, and available on the App Store and Google Play for users who would like to have access to the basic functions of the chain. Within the mobile wallet, users can login via email or manage their 24‑word seed phrase. +Mobile wallets are currently in deployment mode for Testnet Oro and are available on the App Store and Google Play for users who want access to basic chain functions. Within the mobile wallet, users can log in via email or manage their 24-word seed phrase.
12-12: Use “App Store,” not “Apple Store.”Apple’s marketplace is called the App Store; adjust the label for correctness and consistency with Line 2/Line 8.
-Apple Store: [https://apps.apple.com/us/app/kii-mobile/id6474740411](https://apps.apple.com/us/app/kii-mobile/id6474740411) +App Store: [https://apps.apple.com/us/app/kii-mobile/id6474740411](https://apps.apple.com/us/app/kii-mobile/id6474740411)
there's double link in "set up a wallet" link, causing double link but its a different link and causing typo
2 participants
After a sentence ends, a period must be added, but there the period is too far away
Type of change