Skip to content

Conversation

@farhan
Copy link
Contributor

@farhan farhan commented Apr 17, 2025

  • 09_skip_control.js
  • 09_play_placeholder.js
  • 09_play_skip_control.js
  • 09_play_pause_control.js
  • 036_video_social_sharing.js
  • 10_commands.js
  • 09_events_bumper_plugin.js
  • 09_save_state_plugin.js
  • 025_focus_grabber.js
  • 09_events_plugin.js

farhan and others added 23 commits April 16, 2025 10:59
* feat: Group ora staff notifications

* fix: Revert default change for ora staff notifications

* fix: update edx-ora2 version
I'm not sure how this worked before but I corrected to use the
`graphviz` directive and then full graphviz graph syntax which seemed to
fix whatever issues it was having.
DEPR: #36429

This change removes the course_sock and related API data.  The UI it
removes is on the Legacy Courseware pages which have also been replaced
and have their own [deprecation ticket](#35803)

Before this can be merged, we will need to update the
frontend-app-learning MFE to no longer consume the
`can_show_upgrade_sock` attribute.

BREAKING CHANGE: CourseHomeMetadata, ProgressTab, OutlineTab and
VerifiedMode APIs will no longer have a `can_show_upgrade_sock`
attribute.
These flags had no effect, so they are being removed:

* new_core_editors.use_advanced_problem_editor
* new_editors.add_game_block_button

Related to: openedx/frontend-app-authoring#1753
Part of: #26275
There was a waffle flag `contentstore.enable_studio_content_api`,
intended to gate the "experimental" REST APIs at
`<CMS_ROOT>/api/contentstore/v{0,1,2}/*`. In practice, these APIs are no
longer experimental: for the past few named releases, they have been
enabled in Tutor and used to power the Authoring MFE.

We are making the Authoring MFE default-on in all Open edX sites
starting in Teak, with the legacy authoring frontend slated for removal
by Ulmo. Therefore, we need to remove flag which is gating the REST API.
We are _not_ introducing a temporary opt-out toggle, as we do need feel
it is necessary.

Part of: #36275
* feat: container tag support

* fix: fixes from review

* docs: fix typo
* fix: Enable ORA new submissions course notification preference

* fix: update edx-ora2 version
Co-authored-by: M Umar Khan <umar.khan@A006-01609.local>
In the Annotatable XBlock, the <instructions> element was appearing
twice in the student view:

* Once in "annotatable-instructions" (where it should be).
* Again in "annotatable-content" (where annotations and other content are rendered).

The _render_content method processed and rendered the entire XML
data, including <instructions>, without removing it. The
_extract_instructions method, which is responsible for removing
<instructions>, was not called in _render_content, leading to
duplication.

This fix will:

* Prevents duplicate instructions in the student view.
* Maintains the expected behavior of showing instructions only in
  "annotatable-instructions".
* No impact on existing annotation functionality.
…FC-0083] (#36476)

efactors the content_libraries.api to use LibraryCollectionLocator and LibraryContainerLocator keys, instead of passing separate LibraryUsageKeyV2 keys along with the collection/container locators.

Renames misleading uses of collection_usage_key to collection_locator, including in the content_libraries.api and content.search.api method names and parameters.

This change impacts Developers, but should not affect end users.

This refactoring seems reasonable to do without going through deprecation, given the minimal use of these APIs.
XBlock rendered in an iframe in frontend-app-authoring needs to resize automatically based on component size.

Updates xblock iframe template to post message to parent window on size change.
@farhan farhan force-pushed the convert-video-block-js-to-vanilla branch from f8c2f02 to e05bafd Compare April 17, 2025 07:29
@farhan farhan closed this Apr 17, 2025
@farhan farhan deleted the farhan/vanilla-js-conversion-2 branch April 17, 2025 07:36
@farhan
Copy link
Contributor Author

farhan commented Apr 17, 2025

Closed because of duplication
#36537

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.