feat: allow for quiz exports to handle images #357
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Playwright test results
Details
Failed testsCommon persona › tests/aila-chat/downloads.test.ts › Downloading a completed lesson plan Skipped testsNo persona › tests/auth.test.ts › authenticate through Clerk UI |
There was a problem hiding this comment.
I've done some initial tests in Chrome and getting varying results when trying to export the resources.
- First lesson, new, failed to export warnings on all
- Second lesson, old, selected items downloaded
- First lesson (retry), some downloaded then warnings again
The warnings/failures seem sporadic, some generate/download, others fail. No errors or warnings seen in the console at time of download.
|
@tomwisecodes I think importantly for this work to go in, we need it to be able to handle markdown images inline in questions or answers/distractors: {
question: "Complete the statement. Triangle ABC and triangle XYZ are ____________. ",
answers: ["similar as the three interior angles are the same."],
distractors: [
"congruent as the three interior angles are all the same.",
"neither similar nor congruent.",
],
} |
|
| documentId: string; | ||
| data: Data; | ||
| }) { | ||
| // The method here is too locate the placeholder in the documents, delete it, and insert the image. |
There was a problem hiding this comment.
| // The method here is too locate the placeholder in the documents, delete it, and insert the image. | |
| // The method here is to locate the placeholder in the documents, delete it, and insert the image. |
|
Closing in favour of #400 |
Description
This allows for all of our doc exports to be able to handle export images. This work precedes the maths quiz questions work which will call on oak maths questions where the questions and answers are often in image form. The logic may also be reused in the future for adding images to slides.
How to test:
In order to test this run through the download process for a quiz and it should work with no changes