Skip to content

Small changes to the hashed-password FAQ #2533

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jan 14, 2021
Merged

Conversation

SPGoding
Copy link
Contributor

@SPGoding SPGoding commented Dec 30, 2020

See also #2225 (comment)

  • Changed echo to printf to avoid newlines.
  • Added an example to make it clear which password should be changed to hashed-password.
  • Removed the double quotes from the sentence Of course replace `"thisismypassword"` with your actual password. to avoid potential confusion.

@SPGoding SPGoding changed the title Change to use printf in the doc Small changes to the hashed-password FAQ Dec 30, 2020
@SPGoding
Copy link
Contributor Author

Oof, just noticed #2503 🤔

@JayMBS
Copy link

JayMBS commented Jan 11, 2021

works great. tyvm!

@code-asher code-asher merged commit eae285c into coder:master Jan 14, 2021
@code-asher
Copy link
Member

Thank you! ❤️

@SPGoding SPGoding deleted the doc/hash branch January 14, 2021 19:55
@nhooyr nhooyr added this to the v3.8.1 milestone Jan 18, 2021
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.

5 participants