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

Version control module #105

Merged
merged 19 commits into from
Sep 3, 2024
Merged

Version control module #105

merged 19 commits into from
Sep 3, 2024

Conversation

JaroCamphuijsen
Copy link
Collaborator

This PR:

@JaroCamphuijsen
Copy link
Collaborator Author

This PR also updates some thumbnails, as I didn't want to open a new PR just for these tiny changes.

@JaroCamphuijsen JaroCamphuijsen requested a review from raar1 August 27, 2024 08:25
@JaroCamphuijsen JaroCamphuijsen marked this pull request as ready for review August 27, 2024 08:25
Copy link
Collaborator

@raar1 raar1 left a comment

Choose a reason for hiding this comment

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

The "Documents are..." slide doesn't work consistently with loading the fragment. I would suggest getting rid of the fragment tags for now as there might be some clash with the styling.

The key points slide is missing a final bullet point. Or is that intentional?

Everything else looks really nice!

@JaroCamphuijsen JaroCamphuijsen merged commit f0eddad into main Sep 3, 2024
1 check passed
@JaroCamphuijsen
Copy link
Collaborator Author

#117 is supposed to pick up where this PR ends

@JaroCamphuijsen JaroCamphuijsen deleted the version-control-module branch September 3, 2024 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Version control module
3 participants