Skip to content

Conversation

@raboof
Copy link
Member

@raboof raboof commented Jun 26, 2025

Instead of showing people initialPassword and then telling them not to use it, just show initialHashedPassword directly.

@raboof raboof requested a review from a team as a code owner June 26, 2025 12:25
@fricklerhandwerk
Copy link
Collaborator

The reason we did it that way it's because it's much easier to follow manually. More outside interactions we can't control here and more visual noise. Fresh learners are very sensitive to that, and a typo and another round of interactions risks breaking the flow and distracting even further. The tutorial is not supposed to smoothly transfer to production uses, we need many more steps to get people there.

Instead of showing people `initialPassword` and then telling them not to
use it, just show `initialHashedPassword` directly.
@raboof raboof force-pushed the nixos-configuration-on-vm-initialHashedPassword branch from f1f0c02 to 43fa0fc Compare June 26, 2025 14:19
@raboof
Copy link
Member Author

raboof commented Jun 26, 2025

The tutorial is not supposed to smoothly transfer to production uses, we need many more steps to get people there.

That's completely reasonable of course

The reason we did it that way it's because it's much easier to follow manually. More outside interactions we can't control here and more visual noise. Fresh learners are very sensitive to that, and a typo and another round of interactions risks breaking the flow and distracting even further.

That's true, though the current flow of "Here is how you X but actually don't do it that way" is also somewhat off-putting for new learners.

I don't have a good "best of both worlds" suggestion though, so I'll close this for now.

@raboof raboof closed this Jun 26, 2025
@fricklerhandwerk
Copy link
Collaborator

Well we could emphasize just that, that it's for demonstration purposes in order to focus on the essentials, and that working with secrets properly is more involved.

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.

2 participants