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

✨(classroom) attendences analytics #2499

Merged
merged 5 commits into from
Nov 13, 2023

Conversation

AntoLC
Copy link
Contributor

@AntoLC AntoLC commented Nov 8, 2023

Purpose

Feature request: #2492

Proposal

  • adapt ts interfaces
  • calculate attendences analytics
  • display attendences analytics
  • tests
scrnli_11_8_2023_5-07-28.PM.webm

@AntoLC AntoLC self-assigned this Nov 8, 2023
@AntoLC AntoLC linked an issue Nov 8, 2023 that may be closed by this pull request
@AntoLC AntoLC force-pushed the feature/anthony/classroom-attendees branch 3 times, most recently from 222d93d to e0f309f Compare November 8, 2023 16:32
@AntoLC AntoLC marked this pull request as ready for review November 8, 2023 16:37
@AntoLC AntoLC requested review from kernicPanel and lunika and removed request for kernicPanel and lunika November 8, 2023 16:37
CHANGELOG.md Outdated Show resolved Hide resolved
@AntoLC AntoLC force-pushed the feature/anthony/classroom-attendees branch from ee38ad3 to 48527e2 Compare November 9, 2023 11:46
@AntoLC
Copy link
Contributor Author

AntoLC commented Nov 9, 2023

The changes talked during the daily.

scrnli_11_9_2023_12-47-47.PM.webm

@AntoLC AntoLC requested review from kernicPanel and lunika and removed request for kernicPanel and lunika November 9, 2023 11:51
@AntoLC AntoLC force-pushed the feature/anthony/classroom-attendees branch 2 times, most recently from 1b43eba to 3f01ddf Compare November 10, 2023 14:50
@AntoLC
Copy link
Contributor Author

AntoLC commented Nov 10, 2023

scrnli_11_10_2023_3-58-04.PM.webm

description: 'Message displayed if the live has no participant yet',
id: 'components.DashboardLiveControlPane.tab.DashboardLiveTabAttendance.DashboardLiveTabAttendanceWaiting.noParticipants',
},
});

export const DashboardLiveTabAttendanceWaiting = ({}) => {
interface TabAttendanceWaitingProps {
type: 'live' | 'classroom';
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
type: 'live' | 'classroom';
type: 'webinar' | 'classroom';

Copy link
Contributor Author

@AntoLC AntoLC Nov 13, 2023

Choose a reason for hiding this comment

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

@@ -1,5 +1,5 @@
import { Nullable, colorsTokens } from 'lib-common';
import { Box, BoxProps } from 'lib-components';
import { Breakpoints, Nullable, colorsTokens } from 'lib-common';
Copy link
Member

Choose a reason for hiding this comment

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

In the commit title, replace devise by device

Copy link
Contributor Author

@AntoLC AntoLC Nov 13, 2023

Choose a reason for hiding this comment

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

@@ -180,6 +180,9 @@ describe('<ClassroomAttendance />', () => {
{
intlOptions: {
locale: 'fr',
messages: {
'components.ClassroomAttendance.durationLabel': 'Durée',
Copy link
Member

Choose a reason for hiding this comment

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

nice improvement

Copy link
Member

@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.

Nice work!

@AntoLC AntoLC force-pushed the feature/anthony/classroom-attendees branch 3 times, most recently from 16981c1 to be003c0 Compare November 13, 2023 09:03
@AntoLC AntoLC requested a review from lunika November 13, 2023 09:10
Add sessions interface to classroom models.
Move DashboardLiveTabAttendanceWaiting to lib-components.
The component is renamed TabAttendanceWaiting, we refactorize
it to be more generic and reusable for the classroom package.
We create a new tab in the classroom dashboard to show the
attendance of the students in the classroom.
On the small device the margin was too big and the content was not
displayed correctly. We reduce it depend the devise size, plus we
remove some global margin.
We can now pass more options to the wrapInIntlProvider function. This
will allow us to test easily translated texts.
@AntoLC AntoLC force-pushed the feature/anthony/classroom-attendees branch from be003c0 to ddd6145 Compare November 13, 2023 14:20
@AntoLC AntoLC enabled auto-merge (rebase) November 13, 2023 14:21
@AntoLC AntoLC merged commit fa6e449 into master Nov 13, 2023
33 of 34 checks passed
@AntoLC AntoLC deleted the feature/anthony/classroom-attendees branch November 13, 2023 14:33
lunika added a commit that referenced this pull request Nov 29, 2023
Added

- Meta description and meta title on the website from the API (#2516)
- Retrieve BBB learning analytics and send them through API
- Classroom attendance analytics (#2499)
- Add a language picker for the invite link on the website (#2504)

Changed

- Replace grommet DropButton component (#2513)
- On live, can now share resource when one is already shared (#2512)
- Remove persistency on token from invite link (#2505)
- Replace grommet Cards / Footer/ Anchor / Tip / Nav (#2503)
- Refacto widgets SharedLiveMedia (#2504)
- Replace grommet Button (#2453)
- Replace grommet Box (#2484)
- Replace grommet TextArea (#2500)
- Update psycopg to version 3
- Update node to version 20
- Replace pylti with oauthlib for LTI request validation
- Replace faker with @faker-js/faker (frontend)

Fixed

- Blocking error when shared document is deleted (#2504)
- Improve render sharing picture in live (#2508)
lunika added a commit that referenced this pull request Nov 29, 2023
Added

- Meta description and meta title on the website from the API (#2516)
- Retrieve BBB learning analytics and send them through API
- Classroom attendance analytics (#2499)
- Add a language picker for the invite link on the website (#2504)

Changed

- Replace grommet DropButton component (#2513)
- On live, can now share resource when one is already shared (#2512)
- Remove persistency on token from invite link (#2505)
- Replace grommet Cards / Footer/ Anchor / Tip / Nav (#2503)
- Refacto widgets SharedLiveMedia (#2504)
- Replace grommet Button (#2453)
- Replace grommet Box (#2484)
- Replace grommet TextArea (#2500)
- Update psycopg to version 3
- Update node to version 20
- Replace pylti with oauthlib for LTI request validation
- Replace faker with @faker-js/faker (frontend)

Fixed

- Blocking error when shared document is deleted (#2504)
- Improve render sharing picture in live (#2508)
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.

Classroom attendees frontend
3 participants