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

Fixes Coach > Lessons - 'Recipients' drop-down not working #12911

Merged
merged 2 commits into from
Dec 10, 2024

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