Skip to content
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

Moving to the new top down make-link gitbuilding syntax #1

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

julianstirling
Copy link
Collaborator

@julianstirling julianstirling commented Apr 15, 2024

Compiled Documentation

De-duplication

Note that there is now considerably less duplication of information. As the shelves that are used no longer need to be explicitly mentioned in the configuration page, or the 3D printing page.

With this syntax a page can be made for each possible shelf. Only the final construction and the wiring paged need to be written for the specific configuration. All prior pages will be updated/append automatically.

The configuration page

Note that the configuration page can now use a page list to list the tagged shelves, the printing step link, and all the shelf step links are no longer needed:
image

This information is still on the page. And the page ordering is still correct in the navigation:
image

The 3D printing page

The parts that are needed to 3D print are now automatically added to a template page. This not only reduces the need to duplicated this information. It also allows the material quantity to be passed through with the STL and totalled on the page, for a more robust bill of materials:

image

With the final page rendering as:
image


Note that neither the 3D printing page nor the configuration page should need changing for different Nimble configurations.

GitBuilding changes

This MR relies on the new master branch of GitBuilding. Specifically on the make-link syntax added in MR !194 and the make-link templates for manufacturing processes added in MR !195.

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.

1 participant