Skip to content

Conversation

Copy link

Copilot AI commented Jan 5, 2026

QuML player integration was failing in collection-editor due to missing form configuration input, while working correctly in questionset-editor.

Changes

qumlplayer-page.component.ts

  • Added @Input() leafFormConfig and FrameworkService injection
  • Implemented framework data fetching and form field population (fetchFrameWorkDetails, setFieldsTerms, setFormDefaultValues)
  • Updated player configuration to use outcomeDeclaration: {maxScore: {defaultValue}} format instead of direct maxScore property
  • Changed showSolutions/showFeedback from strings to booleans

qumlplayer-page.component.html

  • Replaced 67 lines of hard-coded form fields with <sb-dynamic-form [config]="questionFormConfig">

editor.component.html

  • Added [leafFormConfig]="leafFormConfig" binding to lib-qumlplayer-page

Example

Before:

<lib-qumlplayer-page 
  [questionMetaData]="selectedNodeData" 
  [questionSetHierarchy]="collectionTreeNodes?.data">
</lib-qumlplayer-page>

After:

<lib-qumlplayer-page 
  [leafFormConfig]="leafFormConfig"
  [questionMetaData]="selectedNodeData" 
  [questionSetHierarchy]="collectionTreeNodes?.data">
</lib-qumlplayer-page>

This aligns collection-editor with the working questionset-editor implementation (release-8.1.0).

Original prompt

i want you to help me to fix integrate of QuML player in editor component. it is working fine in https://github.com/Sunbird-inQuiry/editor/blob/release-8.1.0/projects/questionset-editor-library/src/lib/components/editor/editor.component.html

but not woking in https://github.com/Sunbird-Knowlg/sunbird-collection-editor/blob/release-8.1.0/projects/collection-editor-library/src/lib/components/editor/editor.component.html


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Fix integration of QuML player in editor component Add leafFormConfig input to QuML player component for dynamic form rendering Jan 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants