-
Notifications
You must be signed in to change notification settings - Fork 71
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
initial changes to documentation menu structure #1682
initial changes to documentation menu structure #1682
Conversation
Er.. I tried to make a pull request against your branch but instead I seem to have pushed the change. Please feel free to revert if you don't like the suggestion to add the DIG to the placeholder page. It was meant as just a suggestion 😅 I did a test build with these changes and everything looks good! Since I now have a commit on this one, we'll need some one else to merge. |
Renamed menu entry "Add a New User" to "Create and Manage User Accounts" to be in line with Melissa's earlier PR
Tagging @jrubin6 since he is name-dropped in the PR too :) |
@@ -116,7 +201,8 @@ nav: | |||
- 'Resizing a VM': 'technical-documentation/resizing_vm.md' | |||
- 'Checking Coding Standards': 'technical-documentation/checking-coding-standards.md' | |||
- 'Contributing Workflow': 'contributing/contributing-workflow.md' | |||
- 'Editing Documentation': 'contributing/editing-docs.md' |
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.
This is @manez 's new tutorial for people who haven't made a PR before, can we keep it in?
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.
Good eye, I must have lost that one with all the reshuffling. I'll push an update in a bit
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.
Looks good! @Islandora/8-x-committers Can someone swoop in for the merge, since I technically am on the commit list for this PR?
Purpose / why
Evolution of Islandora documentation structure, facilitate gap identification
What changes were made?
Verification
check for unclear page titles, add placeholder entries
Interested Parties
Checklist