Closed Abishekcs closed 1 month ago
Overall, this looks good. Happy to merge once the new setup code is moved out of the presenter.
I have made the necessary changes and moved the logic to load the wiki expert to the controller.
https://github.com/user-attachments/assets/901990dd-7ef2-4d01-a98c-64fa271c024f
@ragesoss, I have made the required changes. Could you please take a look? Thank you again for your review!"
https://github.com/user-attachments/assets/d426a667-ef60-483a-a36b-461812c26a5a
5952
What this PR does
This PR forks the
tagged_courses view
from the existing campaign programs view to allow for further customizations specific to tagged courses. The main feature added in this PR is a new column that displays theWiki Expert(s)
assigned to each course and to avoid N+1 queries on how Wiki Experts are retrieved for each course.Addresses the
first point
of the issue regarding the addition of a Wiki Expert column in the tagged_courses view. This change is part of a larger enhancement of the tagged_courses view used by Wiki Education admin to manage courses based on tags.Screenshots
Before:
https://github.com/user-attachments/assets/d49ad091-f206-47d4-a4a7-f4118a0ab23c
After:
https://github.com/user-attachments/assets/6377a14d-25f2-42f5-bcc1-0213faaa48e8