Skip to content

Conversation

AllanOXDi
Copy link
Member

@AllanOXDi AllanOXDi commented Dec 5, 2024

Summary

This PR fixes Coach > Lesson where the 'Recipients' dropdown values does not show.

please not that when you navigate to Coach > Quizzes and select a value from the 'Recipients' drop-down you will either see a scroll-bar or the contents of the drop-down are not fully visible is an issue with KSelect which is being track by this learningequality/kolibri-design-system#690

References

closes #12898

Reviewer guidance

Navigate to coach > lesson and filter learners by names or entire class

@github-actions github-actions bot added APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) DEV: frontend labels Dec 5, 2024
@AllanOXDi AllanOXDi requested review from LianaHarris360, marcellamaki and pcenov and removed request for LianaHarris360 December 5, 2024 13:42
@AllanOXDi AllanOXDi marked this pull request as ready for review December 5, 2024 13:43
@AllanOXDi AllanOXDi changed the title Fixes Coach > Lessons - The 'Recipients' drop-down not working Fixes Coach > Lessons - 'Recipients' drop-down not working Dec 5, 2024
Copy link
Contributor

github-actions bot commented Dec 5, 2024

@pcenov
Copy link
Member

pcenov commented Dec 6, 2024

Hi @AllanOXDi - I confirm that now the recipients are displayed in the 'Recipients' drop-down at Coach > Learners, however as I have mentioned in #12898 the drop-down is either causing a scroll-bar at the frame or is being displayed upwards. Also the 'Recipients' drop-down is not working at Coach > Learners as well:

2024-12-06_12-24-07.mp4

Let me know if these issues can be addressed here or I should file follow-up issues. Thanks!

@AllanOXDi
Copy link
Member Author

thanks @pcenov . I highly think the other part you raised is related to this. We can file a follow up issue and come back to it once it's fixed.

@pcenov
Copy link
Member

pcenov commented Dec 6, 2024

Hi @AllanOXDi if the issue with the KSelect’s dropdown being displayed above the input is tracked in learningequality/kolibri-design-system#690 then we don't need to file another issue for it, right?
Will you be fixing here the issue with the 'Recipients' drop-down not working at Coach > Learners - looking closely at it, it probably should be removed or changed to something else actually?

@AllanOXDi
Copy link
Member Author

Will you be fixing here the issue with the 'Recipients' drop-down not working at Coach > Learners?

Yes @pcenov, Let me do so now. Thanks

@pcenov
Copy link
Member

pcenov commented Dec 6, 2024

I think someone has added that 'Recipients' drop-down there by mistake, so it should first be clarified why is that drop-down there and what should be the actual filter on that page. On my end I'll approve this PR as it addresses the main issue.

@ozer550
Copy link
Member

ozer550 commented Dec 6, 2024

Hi @AllanOXDi if the issue with the KSelect’s dropdown being displayed above the input is tracked in learningequality/kolibri-design-system#690 then we don't need to file another issue for it, right? Will you be fixing here the issue with the 'Recipients' drop-down not working at Coach > Learners - looking closely at it, it probably should be removed or changed to something else actually?

Hi @pcenov, I have fixed the filtering part for Coach->Learners in the PR that I am currently working on related with csv export functionality mentioned in the issue here. Could you review it there once I open the PR.

@pcenov
Copy link
Member

pcenov commented Dec 6, 2024

Certainly @ozer550 - thanks for letting me know!

Copy link
Member

@LianaHarris360 LianaHarris360 left a comment

Choose a reason for hiding this comment

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

The code logic makes sense and look good to me, thanks Allan!

@LianaHarris360 LianaHarris360 merged commit d5d8c42 into learningequality:develop Dec 10, 2024
39 of 40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) DEV: frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0.18 - Coach > Lessons - The 'Recipients' drop-down is not working
4 participants