ItemBankBlock <- Problem Bank UI #18
commitlint.yml
on: pull_request
commitlint
/
commitlint
25s
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
|