Skip to content

Conversation

@lawik
Copy link
Contributor

@lawik lawik commented May 1, 2025

To whoever is interested. I made an outline for what I think the docs need in terms of providing a clear path to success and step-by-step instruction.

I've also added a draft for the Shared Secret/Quickstart part of it.

Not precious about any of it, but I think this is the type of tutorial that is missing based on questions we've had. There are also needs to explain a bunch of stuff around CAs and whatnot. But this would get people set up I reckon.

Have logged a few issues in the CLI repo that could help cut down on some noise in the guide as well.

@lawik lawik marked this pull request as ready for review May 2, 2025 10:23
@lawik
Copy link
Contributor Author

lawik commented May 2, 2025

The outline document is not actually part of the documentation and can be removed later. Doesn't need to be proof-read or anything :)

to use a NervesKey with the `:primary` key slot and default I2C bus.

```elixir
config :nerves_hub_link,

Choose a reason for hiding this comment

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

You may want to explicitly add connect: true to this example. Depending on how a users config is setup they may put connect: false in a previous config and then the device will not connect.... ask me how I know ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fun :)
I'd like the config to be minimal, there are many defaults. But that's an interesting gotcha.. might add a note on it as an option.

@lawik
Copy link
Contributor Author

lawik commented May 2, 2025

Note to self: nh firmware publish probably already signs the firmware. Saves a step.

Copy link

@nshoes nshoes left a comment

Choose a reason for hiding this comment

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

This is wildly helpful, thanks Lars! FWIW I read through everything and it LGTM.

@lawik lawik merged commit 699b809 into main May 6, 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.

4 participants