Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 14 15:18
    cahirodoherty-learningpool assigned #2973
  • Oct 13 21:13
    chucklorenz opened #3247
  • Oct 13 14:17
    kirsty-hames review_requested #299
  • Oct 13 14:17
    kirsty-hames review_requested #299
  • Oct 13 13:44

    oliverfoster on 1469

    (compare)

  • Oct 13 13:44

    oliverfoster on master

    issue-1469: extended tutor issue-1469: indentation to spac… issue-1468: change default feed… and 17 more (compare)

  • Oct 13 13:44
    oliverfoster closed #61
  • Oct 13 13:44
    oliverfoster closed #1469
  • Oct 13 13:44
    oliverfoster synchronize #61
  • Oct 13 13:44
    oliverfoster synchronize #61
  • Oct 13 13:44

    oliverfoster on 1469

    Update less/tutor.less Co-auth… (compare)

  • Oct 13 13:42

    oliverfoster on 1469

    (compare)

  • Oct 13 13:42
    oliverfoster closed #296
  • Oct 13 13:42

    oliverfoster on master

    issue/1468 extended tutor styli… (compare)

  • Oct 13 13:41
    oliverfoster synchronize #296
  • Oct 13 13:41

    oliverfoster on 1469

    Update less/plugins/adapt-contr… (compare)

  • Oct 13 13:05
    oliverfoster review_requested #61
  • Oct 13 13:05
    oliverfoster review_requested #296
  • Oct 13 13:05
    oliverfoster review_requested #296
  • Oct 13 13:05
    oliverfoster review_requested #296
Cahir O'Doherty
@cahirodoherty-learningpool
Thanks @tomgreenfield for pinpointing the problem so quickly
Cahir O'Doherty
@cahirodoherty-learningpool
happy for me to get that merged and tagged?
tomgreenfield
@tomgreenfield
cheers cahir
Kevin Doherty
@kevindoherty30
Hi All,
We are currently testing f/w 5.14.0 and looking to test out adaptlearning/adapt_framework#3142, although the MCQ/GMCQ updates haven't yet been tagged in a released version of the components. Does anyone know when these plugin updates might get released?
tomgreenfield
@tomgreenfield
i can release those today for you kevin
Kevin Doherty
@kevindoherty30
That would be great, thank you @tomgreenfield
ethan-lp
@ethan-lp
Hey guys, just wanted to query an issue we've seen from Customers today. When assessment feedback is presented to a user placeholders such as {{{score}}} and {{{maxScore}}} are only returning 0. Just wanted to check if we could be missing something regarding config?
Oliver Foster
@oliverfoster
need to know which version of the fw and assessment you're using @ethan-lp
Oliver Foster
@oliverfoster
image.png
current version works
ethan-lp
@ethan-lp
@oliverfoster Thank you. FW is 5.14.0 and Assessment is 4.4.0. So I'll need to dig a little further I think
Oliver Foster
@oliverfoster
:+1:
See if it works using a vanilla course first?
Have you got branching / randomisation on?
ethan-lp
@ethan-lp
Yeah I checked that out already with a vanilla course and only OS plugins. Avoided branching, and with randomisation on or off still no luck
Oliver Foster
@oliverfoster
cool
was it broken in os?
ethan-lp
@ethan-lp
Yes it was also breaking for me with that set up, when I debug through adapt-assessmentArticleModel.js the '_getScore' and '_getMaxScore' are only ever returning 0 for me
Oliver Foster
@oliverfoster
:+1:
will have a look in a minute
ethan-lp
@ethan-lp
Thank you @oliverfoster
eleanor-heath
@eleanor-heath

model.maxScore returns undefined for me in this function within the assessmentArticleModel

_getMaxScore: function() {
const questionComponents = this._getCurrentQuestionComponents();
const maxScore = questionComponents.reduce((maxScore, model) => (maxScore += model.maxScore || 0), 0);
return maxScore;
}

tomgreenfield
@tomgreenfield
does it work with the master branch? the framework hasn't had a tagged release yet with the scoring stuff
Oliver Foster
@oliverfoster
it's because you're using the master assessment which wasn't for 5.14.0
or what tom said
tomgreenfield
@tomgreenfield
i'd prefer to do a release of the framework before the src/core ripper pr goes in. how does that make you feel @oliverfoster?
Oliver Foster
@oliverfoster
warm and fuzzy
tomgreenfield
@tomgreenfield
haha
cool, will get that out today
ethan-lp
@ethan-lp
Thank you both!
Oliver Foster
@oliverfoster
:+1: sorry for the confusion
it's the new item specific scoring stuff
ethan-lp
@ethan-lp
No worries, more so my lack of experience. I couldn't get to the bottom of it on Friday. Appreciate the time and help
Muthu
@muthuselvan86
Hi All, I have a requirement to tailor our course content for a specific customer basically alter the Block/component content. I don't want to create a separate course for this minimum content change instead manage through some flag on same course settings. I am thinking of to create an extension to show /hide the content based on that flag. But is there any better way to achieve this requirement? (I'm using Framework 5.6.1). Appreciate your help!
Oliver Foster
@oliverfoster
nope, that sounds about right, i would duplicate the course and remove the pieces
Muthu
@muthuselvan86
Yea thats quick! The problem will be i end up creating multiple duplicate courses that would be hard to maintain. The approach would be to add a custom text field in block or components JSONS and manage to show the content through flag and template adjustment for specific customer(extension). Thinking of any better approach!.
Oliver Foster
@oliverfoster
you should be able to:
const clientVersion = Adapt.course.get('_clientVersion');
if (!version) return;
Adapt.data.forEach(model => {
  const modelClientVersion = model.get('_clientVersion');
  if (!modelClientVersion) return;
  if (!modelClientVersion.split(' ').includes(clientVersion)) model.set('_isAvailable', false);
});
Which means any model with _clientVersion must match the _clientVersion in course.json
you could then put _clientVersion: 'version1' or _clientVersion: 'version2' in course.json and a block model with _clientVersion: 'version1 version2' would appear
but the same block wouldn't appear if _clientVersion: 'version3' was put in the course.json
tomgreenfield
@tomgreenfield
careful about the String.includes() if the number of versions gets into double digits e.g. version10
Muthu
@muthuselvan86
Wow! Thank you for your time! Nice Solution! Let me summarize if i understand your approach completely. I need to come up with a small extension that should run the above code as soon as the Adapt dataLoaded and set the model availability based on the clientVersion.
Oliver Foster
@oliverfoster
bingo
i would do it at block level probably?
you don't want to end up with empty blocks
Muthu
@muthuselvan86
yea! make sense. Thanks much and let me try this approach.
Oliver Foster
@oliverfoster
if you want it to work in the authoring tool, you'll probably need something a bit more complicated than just a string property on the model, probably a nested object with an _isEnabled flag and the _clientVersion etc
but the principle is still the same
Muthu
@muthuselvan86
Yea I agree! Got your core approach, slowly i can spice it up :)
Oliver Foster
@oliverfoster
:+1:
Safwan Hossain
@safwanhossain92_twitter
Hi there, how do I register the question weights? is there any extensions that i can use?