-
Notifications
You must be signed in to change notification settings - Fork 290
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
📜 Workbook fixes, removes old structure #5952
Conversation
Thank you for contributing! Your pull request is now going on the merge train (choo choo! Do not click update from main anymore, and be sure to allow changes to be pushed to your fork). |
Thanks for the approve Boryana, but I should have made this a draft since it still needs to be translated |
Oh no, I am very sorry. It did appear in my non-draft PR filter, so I reviewed it and even pushed a merge conflict resolving commit. Sorry, I hope this does not mess things up too much. |
The PR was not for review just yet
No my bad! I should have made it draft! |
Ready for @MarleenGilsing to translate <3 |
for more information, see https://pre-commit.ci
@boryanagoncharenko I removed the level 8 workbooks here so we can merge these fixes (should not have been one PR, sorry) will make a new PR with just level 8. |
…kbook-8-content-only
Removes the old workbooks and fixes a date type issue in nl