Skip to content

Move assets using Blockchain Dashboard#372

Merged
kashepavadan merged 4 commits into
mainfrom
transfer-blockchain-app
Jul 6, 2026
Merged

Move assets using Blockchain Dashboard#372
kashepavadan merged 4 commits into
mainfrom
transfer-blockchain-app

Conversation

@kashepavadan

Copy link
Copy Markdown
Contributor

@danisharora099 danisharora099 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Dogfooded this on macOS. I built and launched the app, went through the full setup (generate config → start node → it comes up and connects to peers), and cross-checked every claim in the doc against the logos-blockchain-ui source. The Transfer panel itself is described accurately — the flow, the fields, and the error messages all match what's in the code. A few things I'd fix:

  1. The balance doesn't update on its own after a transfer (this is the main one). The doc says the sender balance "updates on the next balance refresh after the transfer is confirmed," which reads as automatic — but there's no periodic refresh, and a transfer doesn't trigger one. You have to hit Refresh in Accounts manually. Worth rewording so people aren't left staring at a stale balance.
  2. The recipient key isn't validated. "To key (64 hex chars)" is just placeholder text — whatever you type gets passed straight to the module. The troubleshooting line ("verify the recipient key is exactly 64 hex characters") reads like the UI enforces it; it doesn't. Fine as advice, just don't imply the app checks it.
  3. Network mismatch: the app is devnet-only (the config screen only offers Devnet / Custom config — no testnet), but the prereqs link the testnet faucet. Funding a testnet address against a devnet node lands you on the wrong network.
  4. Minor: the "From address" dropdown lists all your known addresses, not only funded ones.

One heads-up that isn't really this doc's fault: I couldn't complete an actual transfer, because the app's node can't sync devnet right now — it starts on a different genesis than the live devnet and never syncs. I confirmed devnet itself is healthy by syncing a CLI node against it at the same time, so this is an app bug, not the doc. But it does mean this journey can't be dogfooded end to end until that's fixed — I'll raise it with the app/R&D folks separately (happy to share the details).

Not blocking the doc on the app bug — the four fixes above are what I'd want in the doc itself.

@weboko weboko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Re-dogfooded after the "danish review" commit. I built logos-blockchain-ui from source per the prerequisite doc (git clone + nix buildblockchain_ui_plugin.so, exit 0) and cross-checked every claim in this doc against the app source. The balance-refresh wording reword (What to expect / result step) reads better now.

Two things still open that I could pin down precisely (inline):

  1. Network mismatch (your point 3 from last round). The app is devnet-only, so the testnet faucet in the prereqs funds the wrong network.
  2. Balance refresh mechanism. The doc points readers at the From address dropdown to refresh, but that control has no refresh; the Refresh button lives in the Accounts panel.

I couldn't drive an actual transfer end-to-end (headless env, no display — and the devnet-sync app bug you already logged), so these are source-verified rather than click-verified. well_known_deployment is hardcoded to "devnet" (BlockchainBackend.cpp:398) and GenerateConfigView.qml only exposes a Devnet radio + Custom, which corroborates your devnet-only finding. FWIW the "live balance"/"any funded wallet address" wording in What to expect is also slightly off (the combo lists all known addresses and the balance is only as fresh as the last manual Refresh) — minor, your call.

Before you start, make sure you have the following:

- A built [Blockchain node app](./build-and-run-logos-blockchain-node-app-ui.md)
- At least one wallet address with a positive balance, for example one funded via the faucet at [`testnet.blockchain.logos.co`](https://testnet.blockchain.logos.co/web/faucet/)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Network mismatch — still open from last round. This links the testnet faucet, but the app is devnet-only:

  • BlockchainBackend.cpp:398 hardcodes deployment.insert("well_known_deployment", "devnet")
  • GenerateConfigView.qml exposes only a Devnet radio (+ Custom) — no testnet option
  • the app's own README.md funds from https://devnet.blockchain.logos.co/web/faucet/

Funding a testnet address against a devnet node puts the wallet on the wrong network. https://devnet.blockchain.logos.co/web/faucet/ returns 200, so:

Suggested change
- At least one wallet address with a positive balance, for example one funded via the faucet at [`testnet.blockchain.logos.co`](https://testnet.blockchain.logos.co/web/faucet/)
- At least one wallet address with a positive balance, for example one funded via the faucet at [`devnet.blockchain.logos.co`](https://devnet.blockchain.logos.co/web/faucet/)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Update after re-dogfooding against the current app: this is now resolved upstream — keep the testnet link, don't apply the devnet suggestion.

logos-blockchain-ui switched from devnet to testnet in #32 "fix(network): default", which landed after my review above. That commit:

  • removes the deployment.insert("well_known_deployment", "devnet"); hardcode in BlockchainBackend.cpp
  • removes the devnetRadio from GenerateConfigView.qml
  • repoints the README faucet + block explorer to testnet.blockchain.logos.co

So the app is on testnet now, this doc's testnet.blockchain.logos.co/web/faucet/ link is correct (200 OK), and the earlier testnet→devnet suggestion would reintroduce the mismatch. I'd resolve this thread.

Also re-verified the rest of the doc end-to-end against the app source on this branch's head (4432c83): the Transfer panel labels (To key (64 hex chars), Amount, Send), the read-only balance / --- placeholder, the result-row copy button, and the three troubleshooting strings (Module not initialized, Call failed., Error: …) all match. The fresh commit's balance-refresh fix is correct — the Refresh button does live in the Accounts panel (AccountsView.qml), not on the From-address combo.

Out of scope for this PR, but flagging for follow-up: the prerequisite build-and-run-logos-blockchain-node-app-ui.md still sends readers to the devnet faucet at Step 3 while the rest of that doc says testnet — worth realigning it to testnet too.

Comment thread docs/blockchain/node-app/move-assets-using-logos-blockchain-app.md Outdated
…p.md

Co-authored-by: Sasha <118575614+weboko@users.noreply.github.com>
@kashepavadan

Copy link
Copy Markdown
Contributor Author

@weboko app should use testnet by default now. Can you check please?

@kashepavadan kashepavadan merged commit dc40935 into main Jul 6, 2026
4 checks passed
@kashepavadan kashepavadan deleted the transfer-blockchain-app branch July 6, 2026 20:03
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