Skip to content

ItemBankBlock <- Problem Bank UI #18

ItemBankBlock <- Problem Bank UI

ItemBankBlock <- Problem Bank UI #18

Triggered via pull request October 21, 2024 16:21
Status Failure
Total duration 1m 26s
Artifacts

commitlint.yml

on: pull_request
commitlint  /  commitlint
25s
commitlint / commitlint
Fit to window
Zoom out
Zoom in

Annotations

1 error
commitlint / commitlint
You have commit messages with errors ⧗ input: feat: new Studio view for rendering a Unit in an iframe [FC-0070] The first attempt at creating a new MFE-driven page for Studio Unit rendering involved rendering each XBlock separately in its own iframe. This turned out to be prohibitively slow because of the many redundant assets and JavaScript processing (e.g. MathJax) that happens for each XBlock component. In order to mitigate some of these issues, we decided to try a hybrid approach where we render the entire Unit's worth of XBlocks at once on the server side in a Studio view + template, and then invoke that from frontend-app-authoring as an iframe. The frontend-app-authoring MFE would still be responsible for displaying most of the interactive UI, but the per-component actions like "edit" would be triggered by buttons on the server-rendered Unit display. When one of those buttons is pressed, the server-rendered UI code in the iframe would use postMessage to communicate to the frontend-app-authoring MFE, which would then display the appropriate actions. To make this work, we're making a new view and template that copies a lot of existing code used to display the Unit in pre-MFE Studio, and then modifying that to remove things like the header/footer so that it can be invoked from an iframe. This entire design is a compromise in order to do as much of the UI development in frontend-app-authoring as possible while keeping XBlock rendering performance tolerable. We hope that we can find better solutions for this later. Authored-by: Sagirov Eugeniy <[email protected]> ✖ header must not start with whitespace [header-trim] ✖ subject may not be empty [subject-empty] ✖ type may not be empty [type-empty] ✖ found 3 problems, 0 warnings ⓘ Get help: https://open-edx-proposals.readthedocs.io/en/latest/oep-0051-bp-conventional-commits.html ⧗ input: Merge: workflow for picking lib content to add to course ✖ type must be lower-case [type-case] ✖ type must be one of [revert, feat, fix, perf, docs, test, build, refactor, style, chore, temp] [type-enum] ✖ found 2 problems, 0 warnings ⓘ Get help: https://open-edx-proposals.readthedocs.io/en/latest/oep-0051-bp-conventional-commits.html ⧗ input: Merge: copy read-only tags when using lib content in a course ✖ type must be lower-case [type-case] ✖ type must be one of [revert, feat, fix, perf, docs, test, build, refactor, style, chore, temp] [type-enum] ✖ found 2 problems, 0 warnings ⓘ Get help: https://open-edx-proposals.readthedocs.io/en/latest/oep-0051-bp-conventional-commits.html