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.

Sign up for GitHub

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

Jump to bottom

Quizz #148

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Quizz #148

wants to merge 3 commits into from

Conversation

MadlenaB
Copy link

Prerequisite Checklist

  • Read the contribution guidelines regarding submitting new changes to the project;
  • Tested your changes against relevant architectures and platforms;
  • Updated relevant documentation (if needed).

Description of changes

@Alex-deVis Alex-deVis requested a review from teodutu December 13, 2024 09:57
@teodutu teodutu added student-contrib Fix or improvement made by a student needs-rendering The PR makes changes to the website that need to be rendered labels Dec 13, 2024
@github-actions GitHub Actions
Copy link

Published at https://cs-pub-ro.github.io/operating-systems/148/

@github-actions github-actions bot added area/infra Update to infrastructure / scripts area/questions Update to questions content area/reading Update to reading content topic/software-stack Related to "Software Stack" chapter kind/improve Improve / Update existing content / item labels Dec 13, 2024
teodutu and others added 2 commits December 14, 2024 17:48
@teodutu
infra: Update openedu_builder version to 0.6.1
a4b2bdc 
This provides us with interactive quizzes intead of a plain webpage.
Also fix broken quiz paths.

Signed-off-by: Teodor Dutu <[email protected]>
@MadlenaB @teodutu
Trying to make the quizzes interactive
af51726 
Signed-off-by: MadlenaB <[email protected]>
@MadlenaB @teodutu
infra: Add support for interactive quizzes
1da3514 
- Use the `quizzify` plugin in `config.yaml`
- Modify `gen-view.py` to fix paths to questions
- Add `.view/` to `.gitignore`

Signed-off-by: MadlenaB <[email protected]>
Signed-off-by: Teodor Dutu <[email protected]>
@teodutu teodutu added needs-rendering The PR makes changes to the website that need to be rendered and removed needs-rendering The PR makes changes to the website that need to be rendered labels Dec 20, 2024
@github-actions GitHub Actions
Copy link

Published at https://cs-pub-ro.github.io/operating-systems/148/

@teodutu
Copy link

teodutu commented Dec 20, 2024 •
edited
Loading

@MadlenaB, I fixed the config and gen-view.py. See my diff for the changes [1]. I mainly just skipped the links to quizzes from being "solved" because the builder plugin handled them separately.

Now it's up to you to fix the CI/CD failures [2]. Squash the commits into one with the same message as my last one and change the Dockerfile to whatever super-linter asks for.

You can see the rendered quizzes here for example [3].

[1] https://github.com/cs-pub-ro/operating-systems/compare/b705a241beba0256e7918dd3de4ce41730bc4e52..1da35141460f6a82bad81781ad24bbdf3459c20e
[2] https://github.com/cs-pub-ro/operating-systems/pull/148/checks
[3] https://cs-pub-ro.github.io/operating-systems/148/Software%20Stack/lab1

@github-actions github-actions bot added the area/guides Update to guides content label Dec 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@teodutu teodutu Awaiting requested review from teodutu

At least 1 approving review is required to merge this pull request.

Assignees

@teodutu teodutu

Labels
area/guides Update to guides content area/infra Update to infrastructure / scripts area/questions Update to questions content area/reading Update to reading content kind/improve Improve / Update existing content / item needs-rendering The PR makes changes to the website that need to be rendered student-contrib Fix or improvement made by a student topic/software-stack Related to "Software Stack" chapter
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MadlenaB @teodutu