Search for courses on A plus front pages#1469
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR implements a search functionality for courses on A+ front pages, allowing users to search for courses by name on both the index page (showing ongoing courses) and the archive page. The search is performed using a query parameter and filters courses based on case-insensitive name matching.
- Adds search form with input field and clear functionality to course pages
- Implements backend filtering logic to handle course name search queries
- Adds translation support for search-related UI text
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| locale/en/LC_MESSAGES/django.po | Adds translation entry for search placeholder text |
| course/views.py | Implements search query filtering in both IndexView and ArchiveView |
| course/templates/course/index.html | Integrates search form into the ongoing courses section |
| course/templates/course/archive.html | Adds search form and no results message to archive page |
| course/templates/course/_search_courses_form.html | Creates reusable search form template with input field and clear button |
| course/static/course/course_search.js | Provides JavaScript functionality for clearing search input |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
1993aa8 to
cb31d87
Compare
cb31d87 to
826d64a
Compare
ihalaij1
approved these changes
Oct 14, 2025
Contributor
ihalaij1
left a comment
There was a problem hiding this comment.
Nice.
I made some fixes and improvements to the UI.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
What?
Implement Search for courses on A plus front pages - index and Archive
Why?
[ANSWER HERE]
How?
[ANSWER HERE]
Fixes #
#579
Testing
Remember to add or update unit tests for new features and changes.
What type of test did you run?
[ADD A DESCRIPTION ABOUT WHAT YOU TESTED MANUALLY]
Did you test the changes in
Think of what is affected by these changes and could become broken
Translation
Programming style
Have you updated the README or other relevant documentation?
Is it Done?
Clean up your git commit history before submitting the pull request!