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

Ch 2, 9, 22 - dead links (Book of Specs) #65

Closed
NielsRenard opened this issue Jan 11, 2020 · 2 comments · Fixed by #66
Closed

Ch 2, 9, 22 - dead links (Book of Specs) #65

NielsRenard opened this issue Jan 11, 2020 · 2 comments · Fixed by #66

Comments

@NielsRenard
Copy link
Contributor

NielsRenard commented Jan 11, 2020

Hi there, still very much enjoying the tutorial!

I noticed some links to the (slide-rs) specs website are broken:

https://github.com/thebracket/rustrogueliketutorial/blob/master/book/src/chapter_2.md
You can learn more about these stores in The Specs Book.

https://github.com/thebracket/rustrogueliketutorial/blob/master/book/src/chapter_3.md
You can learn more about this in the Specs Book.

https://github.com/thebracket/rustrogueliketutorial/blob/master/book/src/chapter_9.md
"We referenced the Book of Specs for how to do this!"

https://github.com/thebracket/rustrogueliketutorial/blob/master/book/src/chapter_22.md
The Specs Book provides a great clue as to how to exclude a component from a join, so we do that (in main.rs)

@NielsRenard NielsRenard changed the title Ch 9 - dead link (Book of Specs) Ch 2, 9, 22 - dead links (Book of Specs) Jan 11, 2020
@NielsRenard
Copy link
Contributor Author

PR ready #66

@NielsRenard
Copy link
Contributor Author

For the chapter 9 specs link: maybe you want to link directly to the "Sort entities based on component value" anchor instead of to the toplevel page, i.e.:
https://specs.amethyst.rs/docs/tutorials/11_advanced_component.html#sorting-entities-based-on-component-value instead of https://specs.amethyst.rs/docs/tutorials/11_advanced_component.html

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 a pull request may close this issue.

1 participant