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) integrate cunningham #2297

Merged
merged 1 commit into from
Aug 1, 2023
Merged

Conversation

AntoLC
Copy link
Contributor

@AntoLC AntoLC commented Jun 23, 2023

Purpose

Add cunningham design system to the apps, load it from lib-common.
Integrate the component DatePicker component to replace the grommet one.

Proposal

  • ➕(frontend) add cunningham design system
  • 💄(frontend) replace to cunningham Datepicker
Marsha.1.webm

@AntoLC AntoLC self-assigned this Jun 23, 2023
@AntoLC AntoLC force-pushed the feature/anthony/cunningham branch 3 times, most recently from 2fe1b58 to 730651f Compare June 29, 2023 12:47
Base automatically changed from feature/anthony/react18 to master June 30, 2023 09:57
@AntoLC AntoLC force-pushed the feature/anthony/cunningham branch 2 times, most recently from 2e36b29 to 04c3a8f Compare June 30, 2023 13:42
@AntoLC
Copy link
Contributor Author

AntoLC commented Jul 10, 2023

We just need this PR on the Cunningham side to be released and all good.

@AntoLC AntoLC force-pushed the feature/anthony/cunningham branch 6 times, most recently from 7ff711d to 0025936 Compare July 18, 2023 09:35
@AntoLC AntoLC marked this pull request as ready for review July 18, 2023 09:55
@AntoLC
Copy link
Contributor Author

AntoLC commented Jul 19, 2023

I changed the retention datepicker as well: fixup! 💄(frontend) replace to cunningham Datepicker

@AntoLC AntoLC requested a review from kernicPanel July 19, 2023 15:16
@AntoLC AntoLC requested review from polyhb and lunika July 19, 2023 15:16
@@ -0,0 +1,39 @@
/*@import '@openfun/cunningham-react/fonts';*/
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi @AntoLC, I wonder why did you comment this? Should it be merged?

Copy link
Contributor Author

@AntoLC AntoLC Jul 20, 2023

Choose a reason for hiding this comment

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

Hi, the font integration didn't seem to work, we have Roboto on our side as well so I just commented the line to come back on it later on.
I removed it, I will open an issue about it when the PR will be merged.
fixup! ➕(frontend) add cunningham design system

Copy link
Member

@lunika lunika left a comment

Choose a reason for hiding this comment

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

In LTI, if the widget is at the dashboard bottom, the date picker open under the uframe and we can't pick the date.

Screenshot from 2023-07-25 14-16-05

@AntoLC AntoLC force-pushed the feature/anthony/cunningham branch from 984a3bd to 9b9ade6 Compare July 31, 2023 10:30
@AntoLC
Copy link
Contributor Author

AntoLC commented Jul 31, 2023

In LTI, if the widget is at the dashboard bottom, the date picker open under the uframe and we can't pick the date.

Screenshot from 2023-07-25 14-16-05

The component doesn't have a position prop, I am not sure how we can change it as it is now. We could eventually open an issue about it.
On the other hand, it is not really blocking because the opening of the datepicker increase the size of the viewport, by scrolling we can have access to the datepicker (see video below).

localhost_8060_development_.webm

@lunika Could you scroll on the datepicker ?

@AntoLC AntoLC requested a review from lunika July 31, 2023 10:37
@AntoLC AntoLC force-pushed the feature/anthony/cunningham branch from 9b9ade6 to d6f8e86 Compare July 31, 2023 10:38
@lunika
Copy link
Member

lunika commented Aug 1, 2023

@AntoLC I can't scroll, the iframe height is not modified and I can't see the datepicker

@AntoLC AntoLC force-pushed the feature/anthony/cunningham branch from d6f8e86 to db8887c Compare August 1, 2023 08:43
@AntoLC
Copy link
Contributor Author

AntoLC commented Aug 1, 2023

@AntoLC I can't scroll, the iframe height is not modified and I can't see the datepicker

From our conversation, I removed the commit about the DatePicker, only stays the Cunningham integration in this PR.

@AntoLC AntoLC force-pushed the feature/anthony/cunningham branch from db8887c to 5e3f680 Compare August 1, 2023 08:46
Add cunningham design system to the apps.
We generate the necessary files from lib-common to serve
the design system to the apps.
@AntoLC AntoLC force-pushed the feature/anthony/cunningham branch from 5e3f680 to b57ff9e Compare August 1, 2023 09:15
@AntoLC AntoLC enabled auto-merge (rebase) August 1, 2023 09:16
@AntoLC AntoLC merged commit 8c5e361 into master Aug 1, 2023
34 checks passed
@AntoLC AntoLC deleted the feature/anthony/cunningham branch August 1, 2023 09:30
lunika added a commit that referenced this pull request Aug 29, 2023
Added

- Create a default playlist for new shibboleth user
- Add Cunningham design system (#2297)
- Lifecycle rule on source bucket to expire uploaded
  object after 21 days.
- Add P2p feature on videojs player
- Webtorrent tracker for P2P video feature
- eslint sort the modules name (#2338)
- Add retention date and s3 lifecycle rules to classroom / video
- Add a command to delete expired classrooms / videos
- Add retention duration on playlist form
- Add retention date widget to video and classroom
- Add a download-video widget directly in the video player
- Configure sentry in the webtorrent application
- Add custom frontend site management
- Manage multiple ingresses
- Manage shibboleth config per site config
- Add a shared media widget directly in the video player
- Add a transcript plugin to the video player
- Add a link to LTI resources to retrieve them in the standalone website
- Add Language Picker in the standalone website (#2366)
- Add xapi statements cache to avoid duplicate statements
- Add variable to override PVC in arnold deployment

Changed

- Upgrade frontend to React 18 (#2280)
- Upgrade to django 4.2
- `DJANGO_STATICFILES_STORAGE` environment variable is replaced by
  `DJANGO_STORAGES_STATICFILES_BACKEND`
- Use alpine for installing mediainfo in lambda docker images
- Add compatibility with docker compose 2
- tslint is replaced by eslint in lti app (#2321)
- refactor videojs id3 tags handling as videojs plugin
- refactor videojs xapi handling as videojs plugin
- upgrade react-query to @tanstack/react-query (v4) (#2340)
- Replace component SortableTable by Cunningham DataGrid (#2311)
- Replace grommet DatePicker by Cunningham DatePicker (#2359)

Fixed

- Video player reset when attributes update (#2300)
- Fix panel closing and resizing on live video player chat (#2310)
- Fix download video button when video is not downloadable
@lunika lunika mentioned this pull request Aug 29, 2023
lunika added a commit that referenced this pull request Aug 30, 2023
Added

- Create a default playlist for new shibboleth user
- Add Cunningham design system (#2297)
- Lifecycle rule on source bucket to expire uploaded
  object after 21 days.
- Add P2p feature on videojs player
- Webtorrent tracker for P2P video feature
- eslint sort the modules name (#2338)
- Add retention date and s3 lifecycle rules to classroom / video
- Add a command to delete expired classrooms / videos
- Add retention duration on playlist form
- Add retention date widget to video and classroom
- Add a download-video widget directly in the video player
- Configure sentry in the webtorrent application
- Add custom frontend site management
- Manage multiple ingresses
- Manage shibboleth config per site config
- Add a shared media widget directly in the video player
- Add a transcript plugin to the video player
- Add a link to LTI resources to retrieve them in the standalone website
- Add Language Picker in the standalone website (#2366)
- Add xapi statements cache to avoid duplicate statements
- Add variable to override PVC in arnold deployment

Changed

- Upgrade frontend to React 18 (#2280)
- Upgrade to django 4.2
- `DJANGO_STATICFILES_STORAGE` environment variable is replaced by
  `DJANGO_STORAGES_STATICFILES_BACKEND`
- Use alpine for installing mediainfo in lambda docker images
- Add compatibility with docker compose 2
- tslint is replaced by eslint in lti app (#2321)
- refactor videojs id3 tags handling as videojs plugin
- refactor videojs xapi handling as videojs plugin
- upgrade react-query to @tanstack/react-query (v4) (#2340)
- Replace component SortableTable by Cunningham DataGrid (#2311)
- Replace grommet DatePicker by Cunningham DatePicker (#2359)

Fixed

- Video player reset when attributes update (#2300)
- Fix panel closing and resizing on live video player chat (#2310)
- Fix download video button when video is not downloadable
lunika added a commit that referenced this pull request Aug 30, 2023
Added

- Create a default playlist for new shibboleth user
- Add Cunningham design system (#2297)
- Lifecycle rule on source bucket to expire uploaded
  object after 21 days.
- Add P2p feature on videojs player
- Webtorrent tracker for P2P video feature
- eslint sort the modules name (#2338)
- Add retention date and s3 lifecycle rules to classroom / video
- Add a command to delete expired classrooms / videos
- Add retention duration on playlist form
- Add retention date widget to video and classroom
- Add a download-video widget directly in the video player
- Configure sentry in the webtorrent application
- Add custom frontend site management
- Manage multiple ingresses
- Manage shibboleth config per site config
- Add a shared media widget directly in the video player
- Add a transcript plugin to the video player
- Add a link to LTI resources to retrieve them in the standalone website
- Add Language Picker in the standalone website (#2366)
- Add xapi statements cache to avoid duplicate statements
- Add variable to override PVC in arnold deployment

Changed

- Upgrade frontend to React 18 (#2280)
- Upgrade to django 4.2
- `DJANGO_STATICFILES_STORAGE` environment variable is replaced by
  `DJANGO_STORAGES_STATICFILES_BACKEND`
- Use alpine for installing mediainfo in lambda docker images
- Add compatibility with docker compose 2
- tslint is replaced by eslint in lti app (#2321)
- refactor videojs id3 tags handling as videojs plugin
- refactor videojs xapi handling as videojs plugin
- upgrade react-query to @tanstack/react-query (v4) (#2340)
- Replace component SortableTable by Cunningham DataGrid (#2311)
- Replace grommet DatePicker by Cunningham DatePicker (#2359)

Fixed

- Video player reset when attributes update (#2300)
- Fix panel closing and resizing on live video player chat (#2310)
- Fix download video button when video is not downloadable
lunika added a commit that referenced this pull request Aug 30, 2023
Added

- Create a default playlist for new shibboleth user
- Add Cunningham design system (#2297)
- Lifecycle rule on source bucket to expire uploaded
  object after 21 days.
- Add P2p feature on videojs player
- Webtorrent tracker for P2P video feature
- eslint sort the modules name (#2338)
- Add retention date and s3 lifecycle rules to classroom / video
- Add a command to delete expired classrooms / videos
- Add retention duration on playlist form
- Add retention date widget to video and classroom
- Add a download-video widget directly in the video player
- Configure sentry in the webtorrent application
- Add custom frontend site management
- Manage multiple ingresses
- Manage shibboleth config per site config
- Add a shared media widget directly in the video player
- Add a transcript plugin to the video player
- Add a link to LTI resources to retrieve them in the standalone website
- Add Language Picker in the standalone website (#2366)
- Add xapi statements cache to avoid duplicate statements
- Add variable to override PVC in arnold deployment

Changed

- Upgrade frontend to React 18 (#2280)
- Upgrade to django 4.2
- `DJANGO_STATICFILES_STORAGE` environment variable is replaced by
  `DJANGO_STORAGES_STATICFILES_BACKEND`
- Use alpine for installing mediainfo in lambda docker images
- Add compatibility with docker compose 2
- tslint is replaced by eslint in lti app (#2321)
- refactor videojs id3 tags handling as videojs plugin
- refactor videojs xapi handling as videojs plugin
- upgrade react-query to @tanstack/react-query (v4) (#2340)
- Replace component SortableTable by Cunningham DataGrid (#2311)
- Replace grommet DatePicker by Cunningham DatePicker (#2359)

Fixed

- Video player reset when attributes update (#2300)
- Fix panel closing and resizing on live video player chat (#2310)
- Fix download video button when video is not downloadable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants