Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

⚡️(frontend) lib-classroom live transpiling #2157

Merged
merged 3 commits into from
Mar 30, 2023

Conversation

AntoLC
Copy link
Contributor

@AntoLC AntoLC commented Mar 29, 2023

Purpose

Improve development process by not having to rebuild our packages after a change.
We will listen our packages from our applications and transpile automatically when a change happen.

Proposal

  • adapt lib_classroom with the new setup
  • remove rollup from lib_classroom
  • plug lib_classroom to standalone and LTI

@AntoLC AntoLC changed the base branch from master to feature/anthony/transpile-on-the-fly March 29, 2023 14:39
@AntoLC AntoLC force-pushed the feature/anthony/lib-classroom/transpile-on-the-fly branch from 4274741 to caeb666 Compare March 29, 2023 15:24
Base automatically changed from feature/anthony/transpile-on-the-fly to master March 29, 2023 15:33
@AntoLC AntoLC force-pushed the feature/anthony/lib-classroom/transpile-on-the-fly branch from caeb666 to 005cb07 Compare March 29, 2023 15:41
Add lib-classroom in lti and standalone site with
the transpiling on the fly.
@AntoLC AntoLC force-pushed the feature/anthony/lib-classroom/transpile-on-the-fly branch from 005cb07 to 8faf5e2 Compare March 29, 2023 15:44
@AntoLC AntoLC self-assigned this Mar 29, 2023
@AntoLC AntoLC added improvement dependencies Pull requests that update a dependency file Frontend labels Mar 29, 2023
@AntoLC AntoLC marked this pull request as ready for review March 29, 2023 15:45
@AntoLC AntoLC requested review from lunika and kernicPanel and removed request for kernicPanel and lunika March 29, 2023 15:45
By including lib-classroom in the standalone webpack, we got relative
imports problems coming from lib-classroom, it is because the
relative import are from the apps perspective.
This commit fixes this issue by using the `@lib-classroom` alias.
We don't need to build lib_classroom anymore, it is transpile during
the apps build.
- review dependencies packages about lib_classroom
- 🔥  all build relative dependencies from lib_classroom
- 🔥 rollup.config.mjs from lib_classroom
- adapt CI config and frontend package
@AntoLC AntoLC force-pushed the feature/anthony/lib-classroom/transpile-on-the-fly branch from 8faf5e2 to fcd0df4 Compare March 30, 2023 07:57
@AntoLC AntoLC requested a review from kernicPanel March 30, 2023 08:13
Copy link
Contributor

@kernicPanel kernicPanel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

@AntoLC AntoLC merged commit 32b2391 into master Mar 30, 2023
@AntoLC AntoLC deleted the feature/anthony/lib-classroom/transpile-on-the-fly branch March 30, 2023 08:52
lunika added a commit that referenced this pull request Apr 19, 2023
Added

- standalone website:
  - Integrate VOD dashboard (#2086)
  - List the lives in the contents section (#2104)
  - Live session model
  - Livesession backend rewrite
  - Add sentry
  - Create a live from the website (#2134)
  - Integrate webinar dashboard (#2135)
- Add a License Manager widget for LTI VOD view
- Add a title to the classroom file dropzone
- Add can_edit property on a serialized video
- Add an attribute to consumer site to deactivate resources in LTI select
- live transpilation on lib-video (#2150)
- live transpilation on lib-classroom (#2157)
- live transpilation on lib-markdown (#2160)
- live transpilation on lib-components (#2161)
- live transpilation on lib-tests (#2163)
- live transpilation on lib-common (#2164)
- Add a widget provider for the classroom creation form
- Allow delete playlist resources
  - FileDepository
  - Classroom
  - Document
  - Markdown
- Allow delete playlist
- add routes related URL:
  - thumbnails
  - timed_text_track
  - shared_live_media
- Add classroom widgets :
  - InfoBar
  - Description
  - Scheduling
  - Invite links
  - Support sharing
  - Recordings
- Add classroom invite link for an instructor
- Add a "Tools & Applications" widget for classrooms

Changed

- Update live sessions API to use nested video ID route
- Move generic widget components from lib-video to lib-components
- Make video dashboard collapsed by default
- improve the dropdown languages positionning in the dashboard (#2138)
- Make video dashboard visible by default, and collapsed when using the
  Moodle atto plugin
- Update live_session api to use mixin to prevent url crafting
- standalone website:
  - put the creating ressource form submit button disabled when the
    form is invalid (#2175)

Fixed

- redirect to error page when VOD is deleted
- Manage disconnection when multiple tabs were open on standalone site
- synchronisation between pages for the VOD description widget
- tooltip position on the website context dashboard (#2136)
- thumbnail not reset correctly on the video player (#2137)
- display title / description when a classroom is not scheduled and not started
- correctly fetch transcript content in TranscriptReader
- remove unused 'initiate-live' permissions
- increase debounce time in classroom description widget
- remove id3 tags upload when live channel is not ready
- add an invitation link for moderators in a launched classroom if available
lunika added a commit that referenced this pull request Apr 19, 2023
Added

- standalone website:
  - Integrate VOD dashboard (#2086)
  - List the lives in the contents section (#2104)
  - Live session model
  - Livesession backend rewrite
  - Add sentry
  - Create a live from the website (#2134)
  - Integrate webinar dashboard (#2135)
- Add a License Manager widget for LTI VOD view
- Add a title to the classroom file dropzone
- Add can_edit property on a serialized video
- Add an attribute to consumer site to deactivate resources in LTI select
- live transpilation on lib-video (#2150)
- live transpilation on lib-classroom (#2157)
- live transpilation on lib-markdown (#2160)
- live transpilation on lib-components (#2161)
- live transpilation on lib-tests (#2163)
- live transpilation on lib-common (#2164)
- Add a widget provider for the classroom creation form
- Allow delete playlist resources
  - FileDepository
  - Classroom
  - Document
  - Markdown
- Allow delete playlist
- add routes related URL:
  - thumbnails
  - timed_text_track
  - shared_live_media
- Add classroom widgets :
  - InfoBar
  - Description
  - Scheduling
  - Invite links
  - Support sharing
  - Recordings
- Add classroom invite link for an instructor
- Add a "Tools & Applications" widget for classrooms

Changed

- Update live sessions API to use nested video ID route
- Move generic widget components from lib-video to lib-components
- Make video dashboard collapsed by default
- improve the dropdown languages positionning in the dashboard (#2138)
- Make video dashboard visible by default, and collapsed when using the
  Moodle atto plugin
- Update live_session api to use mixin to prevent url crafting
- standalone website:
  - put the creating ressource form submit button disabled when the
    form is invalid (#2175)

Fixed

- redirect to error page when VOD is deleted
- Manage disconnection when multiple tabs were open on standalone site
- synchronisation between pages for the VOD description widget
- tooltip position on the website context dashboard (#2136)
- thumbnail not reset correctly on the video player (#2137)
- display title / description when a classroom is not scheduled and not started
- correctly fetch transcript content in TranscriptReader
- remove unused 'initiate-live' permissions
- increase debounce time in classroom description widget
- remove id3 tags upload when live channel is not ready
- add an invitation link for moderators in a launched classroom if available
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file Frontend improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants