Open TomWoodward opened 4 months ago
I'm pleased and excited that you've attempted to add some Import capabilities.
Content Package import is an extremely advanced capability. There are mountains of code associated with CP Import and Item + Test Validation in the Amp-up.io Content Bank.
Thank you for this PR and also for your suggestion about using a Github Discussion. I have done as you've suggested and enabled a Discussion https://github.com/amp-up-io/qti3-item-player-controller/discussions because this is an important topic. Can I interest you in moving this topic over here?
On Fri, May 17, 2024 at 10:15 AM Thomas Woodward @.***> wrote:
I was trying to get more familiar with the runner and test it with different qti content. i'm not sure if this is something you're interested in merging, if you are i will clean up the gross parts.
the singleton pattern is not strictly necessary because the underlying array is shared anyway, but because i was expecting the changes to be shared it seemed like a good idea at the time.
the assessment level import is super naive, but it didn't seem like the project supported the "Test" level features anyway so i left it at putting all the items in a list without any of the other stuff.
i've never worked with vue before and i'm new to qti, i would like to learn more. some of the example content i tried to use did not work so great, but i'm not sure how intended that is based on the limitations of the examples and project support. if you wanted to enable discussions on this repo, or had some other venue for questions, i would love to talk more about it.
You can view, comment on, or merge this pull request online at:
https://github.com/amp-up-io/qti3-item-player-controller/pull/6 Commit Summary
- e101495 https://github.com/amp-up-io/qti3-item-player-controller/pull/6/commits/e1014955947ab517d2a6b03bf9acac4bc5ed45d8 this is becoming more complicated than i thought
- 0fa1198 https://github.com/amp-up-io/qti3-item-player-controller/pull/6/commits/0fa1198df567b6d5d3a638760e5a2a6aeb0e9366 grossly making progress
- 0402557 https://github.com/amp-up-io/qti3-item-player-controller/pull/6/commits/0402557bbf476da85e6241955fd762f3ed6bb6e2 refine import a little
- 4de1302 https://github.com/amp-up-io/qti3-item-player-controller/pull/6/commits/4de13025fbc30ebb1de4b479f24acc5366e5f397 Revert whitespace
File Changes
(4 files https://github.com/amp-up-io/qti3-item-player-controller/pull/6/files)
- M src/helpers/ItemFactory.js https://github.com/amp-up-io/qti3-item-player-controller/pull/6/files#diff-a1904398cc246bddce4a6e3c32defe36cd5108640d7a81d8594bd701e3750d47 (15)
- M src/helpers/TestControllerUtilities.js https://github.com/amp-up-io/qti3-item-player-controller/pull/6/files#diff-cb230ffbecdd4d2dd48aba9e540ddc16267f7687730ae719460011c0e24a4e37 (4)
- M src/helpers/TestFactory.js https://github.com/amp-up-io/qti3-item-player-controller/pull/6/files#diff-b5a8377512a0bc32b3d2f861b85cfdc7c467a7b3d9440af16ab629dd12707810 (29)
- M src/views/Home.vue https://github.com/amp-up-io/qti3-item-player-controller/pull/6/files#diff-2dc54a20fa4ab6845b4b0f252336e107d33a13f5c05d966e5c104770a1b292fd (109)
Patch Links:
- https://github.com/amp-up-io/qti3-item-player-controller/pull/6.patch
- https://github.com/amp-up-io/qti3-item-player-controller/pull/6.diff
— Reply to this email directly, view it on GitHub https://github.com/amp-up-io/qti3-item-player-controller/pull/6, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAG3MLI7NRHWOYQFVTF5PEDZCY3KXAVCNFSM6AAAAABH4P6JJWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGMYDGMRTHA4DIOA . You are receiving this because you are subscribed to this thread.Message ID: @.***>
I was trying to get more familiar with the runner and test it with different qti content. i'm not sure if this is something you're interested in merging, if you are i will clean up the gross parts.
the singleton pattern is not strictly necessary because the underlying array is shared anyway, but because i was expecting the changes to be shared it seemed like a good idea at the time.
the assessment level import is super naive, but it didn't seem like the project supported the "Test" level features anyway so i left it at putting all the items in a list without any of the other stuff.
i've never worked with vue before and i'm new to qti, i would like to learn more. some of the example content i tried to use did not work so great, but i'm not sure how intended that is based on the limitations of the examples and project support. if you wanted to enable discussions on this repo, or had some other venue for questions, i would love to talk more about it.