Adapt_framework: contrib-tutor: Title not display title in feedback

[canstudios-louisem]

Tutor is showing the components title not display title in the feedback notify.

[chetan-hajare]

Hi @canstudios-louisem
We are using the title in the feedback since the beginning of the development of the Adapt framework. The page level progress extension also uses title to represent component in the drawer. If we think to change the feedback title to display title, we might have to consider the page level progress extension too. @brian-learningpool @moloko @oliverfoster @lc-thomasberger Can you please provide your suggestions on this?

[oliverfoster]

i think the feedback should be able to specify a title and any additional properties for each feedback item to pass off to tutor.

[oliverfoster]

need to expand this object to take titles, body text, images etc
https://github.com/adaptlearning/adapt_framework/blob/master/src/course/en/components.json#L424

[tomgreenfield]

Related: https://github.com/adaptlearning/adapt_framework/issues/1734.

[lc-thomasberger]

For PageLevelProgress it is actually good to use title instead of displayTitle (long titles do not work with Drawer), so I think we should stick with this behavior.
For questions a separate title field would be nice. It should default to the component's title if it is not specified.

[canstudios-louisem]

Being able to set the feedback's notify title would fix this issue for us really. Most of the time we just want it to say Feedback or similar.

[oliverfoster]

@canstudios-louisem one for _partlyCorrect, corract and incorrect? Or just a general title per component?

[chetan-hajare]

I would prefer a general title per component. What do you think about this @canstudios-louisem ?

[chetan-hajare]

PR: https://github.com/adaptlearning/adapt_framework/pull/1954

[canstudios-louisem]

Thanks for your work on this @chetan-hajare much appreciated :+1:

[moloko]

should all be done now that https://github.com/adaptlearning/adapt-contrib-mcq/pull/139 is in.