Skip to content

Conversation

@woogieboogie-jl
Copy link
Collaborator

Reflecting remaining minor feedbacks to the readme

README.md Outdated
1. Download [TON Keeper](https://tonkeeper.com/) on iOS or Android
### Mobile (Tonkeeper App)

1. Download [Tonkeeper](https://tonkeeper.com/) on iOS or Android
Copy link
Contributor

Choose a reason for hiding this comment

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

I would make sure that these are the exact steps for the mobile app, because I wrote the specifically for the desktop wallet. Are mobile wallet instructions necessary? Everything else in the starter kit is done on a computer anyway.

Choose a reason for hiding this comment

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

@woogieboogie-jl unless its unavoidable i would recommend using only desktop wallet setup steps. in the actual getting started steps. If you wish to inform users that there are mobile options then just link the mobile wallet in a sentence and leave at that.

Copy link
Collaborator Author

@woogieboogie-jl woogieboogie-jl Dec 29, 2025

Choose a reason for hiding this comment

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

@thodges-gh @zeuslawyer

a38cdd2

  • Now uses only desktop wallet setup steps (excluded mobile setup)
  • Double-checked and made corrections to the TONkeepers desktop wallet setup steps.

Choose a reason for hiding this comment

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

thanks. will let @thodges-gh close out as i've not reviewed as deep as he has.

Copy link

@zeuslawyer zeuslawyer left a comment

Choose a reason for hiding this comment

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

LGTM

@thodges-gh thodges-gh merged commit cc4fd32 into main Jan 8, 2026
4 of 5 checks passed
@thodges-gh thodges-gh deleted the minor-feedback-updates branch January 8, 2026 21:05
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