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

Update register.md #121

Merged
merged 1 commit into from
Feb 25, 2025
Merged

Update register.md #121

merged 1 commit into from
Feb 25, 2025

Conversation

alabulei1
Copy link
Contributor

No description provided.

Copy link
Contributor

juntao commented Feb 25, 2025

Hello, I am a PR summary agent on flows.network. Here are my reviews of code commits in this PR.


Commit bbd310e1fbcd50b7f8bf2fd5bdf70a68fb4ef4d7

Key Changes Summary

  1. Step 1 in Connection Process: Changed from "click 'Connect'" to "click 'Launch App'".
  2. Step 2 in Connection Process: Updated to clarify that you should click Connect after launching the app.
  3. Grammar Correction: Added an article ("an") before "ETH address and keystore and password for your node" for better readability.
  4. Clarity in Device ID Description: Changed from "visible to yourself" to "visible to you" for clarity.
  5. Removed Trailing Newline: Eliminated the trailing newline at the end of the file, ensuring consistent formatting.

Copy link
Collaborator

@JYC0413 JYC0413 left a comment

Choose a reason for hiding this comment

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

LGTM

@JYC0413 JYC0413 merged commit c64418c into main Feb 25, 2025
2 checks passed
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.

3 participants