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

fix: path to head assembly 3d parts #319

Merged
merged 1 commit into from
Jul 6, 2021

Conversation

sethfischer
Copy link
Contributor

Fix path to head Head Assembly 3D parts and make the path a link.

@sethfischer sethfischer force-pushed the fix-head-3d-parts-link branch 3 times, most recently from 1f14e31 to e74e0bd Compare June 29, 2021 20:58
Copy link
Collaborator

@Achllle Achllle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest using a relative link but looks good otherwise

@@ -42,7 +42,7 @@ The head assembly serves as the head and face of our rover. It houses a 16x32 fu


## 3. 3D Printing
There are a few components that need to be 3D printed to make the head assembly. You can find the STL files necessary for these prints in the ”Mechanical/Head Assembly/3D Printed Parts” folder of the repository.
There are a few components that need to be 3D printed to make the head assembly. You can find the STL files necessary for these prints in the [mechanical/head_assembly/3d_printed_parts](https://github.com/nasa-jpl/open-source-rover/tree/master/mechanical/head_assembly/3d_printed_parts) folder of the repository.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
There are a few components that need to be 3D printed to make the head assembly. You can find the STL files necessary for these prints in the [mechanical/head_assembly/3d_printed_parts](https://github.com/nasa-jpl/open-source-rover/tree/master/mechanical/head_assembly/3d_printed_parts) folder of the repository.
There are a few components that need to be 3D printed to make the head assembly. You can find the STL files necessary for these prints in the [mechanical/head_assembly/3d_printed_parts](3d_printed_parts) folder of the repository.

Relative link should work as well

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Achllle Link updated

@sethfischer sethfischer force-pushed the fix-head-3d-parts-link branch from e74e0bd to d62c5f1 Compare July 2, 2021 05:25
Fix path to head Head Assembly 3D parts and make the path a link.
@sethfischer sethfischer force-pushed the fix-head-3d-parts-link branch from d62c5f1 to 5073456 Compare July 3, 2021 23:40
@ericjunkins
Copy link
Collaborator

LGTM

@ericjunkins ericjunkins merged commit 67ec8ea into nasa-jpl:master Jul 6, 2021
@sethfischer sethfischer deleted the fix-head-3d-parts-link branch July 7, 2021 08:04
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.

3 participants