- ๐จ๐ฆCanada Liam McDermott
Liam McDermott โ made their first commit to this issueโs fork.
- @liam-mcdermott opened merge request.
- ๐จ๐ฆCanada Liam McDermott
I created an MR that adds the dependency, and tested it.
Hey cornifex, thanks for the patch. Unfortunatelly it doesn't work for me and I'm still getting the error.
This would be due to patches being applied after composer has resolved dependencies, because of this, adding new dependencies in patches doesn't work.
Instead, if you add this to your project's `composer.json` as the first item under `repositories`:
{ "type": "vcs", "url": "git@git.drupal.org:issue/opigno_scorm-3281986.git" },
And put this under your `composer.json`'s `require` section:
"drupal/opigno_scorm": "dev-3281986-json-schema-dep-missing",
It should fix the issue. Or you can add it as a direct dependency, like saurabh-2k17 suggested, but that won't help test the fix for this issue. If the above works for anyone else, please come and mark this as 'Reviewed and tested by the community'.
To reproduce this issue, install Opigno SCORM without any of the other Opigno modules, then upload a SCORM file to some content, load that content in the browser, then open the browser console and interact with the loaded SCORM content. When moving between steps this error should be shown in the console (it's a JS http request).
- Issue was unassigned.
- Status changed to RTBC
8 months ago 1:53am 3 April 2024 - ๐บ๐ธUnited States kurttrowbridge
Hi! I followed the instructions in #7 while also excluding this module from the standard Drupal repository (see snippet below), and that installed the dependencies successfully and fixed the errors. Marking RTBC. Thanks!
"repositories": [ { "type": "composer", "url": "https://packages.drupal.org/8", "exclude": [ "drupal/opigno_scorm" ] }, ... { "type": "vcs", "url": "git@git.drupal.org:issue/opigno_scorm-3281986.git" } ],