-
Notifications
You must be signed in to change notification settings - Fork 30
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
Update password reset Documentation (#404) #505
Conversation
update toctree for legacy pwd reset new screenshots set up legacy reset vs. new
35da299
to
5436bf6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Hooverdan96 Thanks again for yet another update.
This is, as usual, very nice. I've just got a couple of formatting issues that I think, but I'm not certain, are caused by a formatting hang-over. I.e. prior line ending in " ::" clashing with an otherwise defined console block format. Hence my suggestions re using a regular ":" to end the preceding user-visible text as we are introducing that which was promised. And we now have the working earlier in the same new doc .. code-block:: console
.
I can easily re-test the formatting, post you applying the suggested change at your end, if you have little time to confirm my suspicion regarding these end-format fails.
Co-authored-by: Philip Guyton <[email protected]>
Co-authored-by: Philip Guyton <[email protected]>
yes, I think you were right on that. I had fixed some further up before pushing, but I apparently missed a couple (the black background and somewhat "dim" font color contrast is throwing me off still probably). |
@Hooverdan96 Thanks, and yes that does look to have fixed those console block issues: Cheers. I'll merge and get this published then. |
PR product PRODUCTION published. |
Fixes #404
This pull request's proposal
move old password reset procedure to legacy, update existing page with V4 version, as it's also a hard-coded link to the login screen on the Rockstor WebUI.
Checklist
Note: Link check will continue to fail until currently open PR addressing this is merged.