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

add colophon content #234

Merged
merged 1 commit into from
Jun 21, 2024
Merged

add colophon content #234

merged 1 commit into from
Jun 21, 2024

Conversation

oneilsh
Copy link
Contributor

@oneilsh oneilsh commented Jun 21, 2024

Hey @wibeasley, I polished and migrated the colophon content 🙌

I did a bit of massaging of the text you added, removing mention of the chrome extension and hours per chapter as I thought those were a bit in the weeds, but I left in the checklists business as those were a pretty crucial part of the process IMO. Let me know what you think :) If the chrome extension was really crucial maybe it is worth mentioning for others who want to do similar things.

For some reason that lightbox extension isn't working on the fig in this chapter, but we can fix it up later.

Thank you!

@oneilsh oneilsh requested a review from wibeasley June 21, 2024 19:41
@wibeasley wibeasley merged commit 7f69814 into main Jun 21, 2024
1 check passed
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.

2 participants