Build and run the Logos Blockchain UI App#271
Conversation
cheny0
added
area:blockchain
jzaki
left a comment
jzaki
left a comment
There was a problem hiding this comment.
The build command does not work after following the step to clone the repo.
(testing with nix run, but a lot of deps seem to be building fresh...)
| Before you start, make sure you have the following: | ||
|
|
||
| - A supported OS: Linux x86_64 or macOS | ||
| - [Nix](https://github.com/NixOS/nix-installer) installed with flakes enabled. |
There was a problem hiding this comment.
Good to see --enable-flakes as part of the single install line!
Ideally we could have the single install line here (rather than have the user leave to the NixOS repo), and ensure it doesn't go stale by having it in a script, and add a pre-processing step to the docs to take that command and insert it here.
Not for this PR just an idea issue for such scenarios. Note this is different to the automated testing of tutorials in the tool used by core.
| ## What to expect | ||
|
|
||
| - You have a Logos Blockchain node running and connected to peers. | ||
| - Your wallet holds testnet funds requested from the faucet. |
There was a problem hiding this comment.
To me this list isn't clearly the post-conditions. This line in particular reads like it might be a pre-condition.
Maybe it's a matter of adding something like...
## What to expect
After this tutorial you will:
- Have a ...
Or fleshing out the title: What to expect at the end of this doc
| slug: start-a-logos-blockchain-node-via-the-app-ui | ||
| --- | ||
|
|
||
| # Start a Logos Blockchain node via the app UI |
There was a problem hiding this comment.
I thought this was about using basecamp to install a module to just run a node, but realised later it's about building and running the node front-end.
Consider a title that includes 'build'? Something like: Build and run the Logos Blockchain node front-end app
There was a problem hiding this comment.
I would prefer to keep as-is since the goal of the tutorial is to have a blockchain node, and the way to get one is via the app. The goal is not building or running the app per se.
There was a problem hiding this comment.
The instruction spell out how to build the app. Following the instructions it spent 1hr building before I could use it.
| 1. Build the standalone app: | ||
|
|
||
| ```sh | ||
| nix build '.#app' |
There was a problem hiding this comment.
Does not find .#app target. nix build builds a lib, and nix run seems to be building a lot of things...
Not sure if there's a config here (or was in a past commit) where '.#app' worked quicker?
There was a problem hiding this comment.
FYI, nix run Has been running for over 1hr and not yet finished
Note: doing nix run is specified in the blockchain-ui repo's readme as how to run it.
There was a problem hiding this comment.
@Khushboo-dev-cpp check out this issue with the blockchain ui
There was a problem hiding this comment.
Not necessarily an issue?
It did eventually finish. It just took a lot of time and memory to build.
kashepavadan
changed the title
|
@jzaki check it out now |
|
Hi Guys, I tested a similar Logos build flow on macOS (Apple Silicon) and encountered a few onboarding issues around Nix configuration that may be worth documenting. Initially: nix build '.#app'failed because flakes were not enabled. After retrying with: nix --extra-experimental-features 'nix-command flakes' build '.#app'Nix prompted me to trust the Logos cache: but then emitted: From a first-time contributor pov, it wasn't clear whether these warnings are expected, whether additional Nix configuration is required, or whether the build can safely proceed without the cache. It may be helpful to add a troubleshooting note explaining the expected behavior for users who have not previously configured Nix with the Logos cache and flakes enabled. |
4 participants
No description provided.