Skip to content

Implementing Advanced Project Search#278

Open
CyberHunter12-ui wants to merge 8 commits into
komalharshita:mainfrom
CyberHunter12-ui:resolved-conflits-data/projects.json
Open

Implementing Advanced Project Search#278
CyberHunter12-ui wants to merge 8 commits into
komalharshita:mainfrom
CyberHunter12-ui:resolved-conflits-data/projects.json

Conversation

@CyberHunter12-ui
Copy link
Copy Markdown

Summary [required]

Related Issue [required]

Closes #

Type of Change [required]

  • Bug fix — resolves a broken behaviour
  • Feature — adds new functionality
  • Data — adds new projects to data/projects.json
  • Documentation — updates docs, README, or code comments only
  • Style — CSS or visual changes only, no logic change
  • Refactor — restructures code without changing behaviour
  • Test — adds or updates tests

What Was Changed [required]

File Change made
utils/recommender.py Added clear_cache() function
tests/test_basic.py Added test for cache invalidation

How to Test This PR [required]

  1. Clone this branch: git checkout your-branch-name
  2. Install dependencies: pip install -r requirements.txt
  3. Run the app: python app.py
  4. Open http://127.0.0.1:5000 and...
  5. Run the tests: python tests/test_basic.py

Expected test output:

27 passed, 0 failed out of 27 tests

Test Results [required]

paste output here

Screenshots (if UI change)

Before After
screenshot screenshot

Self-Review Checklist [required]

  • I have read CONTRIBUTING.md and followed all guidelines
  • My branch name follows the convention: feat/, fix/, docs/, data/, style/, test/
  • I have run python tests/test_basic.py and all 27 tests pass
  • I have run flake8 . locally and there are no errors
  • I have not introduced any print() or console.log() debug statements
  • Every new function I wrote has a docstring
  • I have not modified files outside the scope of the linked issue
  • If I changed the UI, I tested it at 375px (mobile) and 1280px (desktop)
  • If I added a project to the dataset, it has all required JSON fields

Notes for Reviewer

@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

Someone is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

@CyberHunter12-ui
Copy link
Copy Markdown
Author

image @komalharshita It passed all the test cases the above is the screenshot for the same.

Have a look at the pull request and the updations.

@komalharshita
Copy link
Copy Markdown
Owner

@CyberHunter12-ui fill in the PR template

@CyberHunter12-ui
Copy link
Copy Markdown
Author

Summary [required]

This pull request implements an advanced project search feature to significantly improve project discoverability on the platform. It introduces a clear_cache() utility within the recommendation engine to ensure that search queries return fresh, relevant results without stale data bottlenecks. Additionally, new project data has been integrated into the central dataset, backed by automated tests verifying the cache invalidation logic.

Related Issue [required]

Closes #

Type of Change [required]

  • Feature — adds new functionality
  • Data — adds new projects to data/projects.json
  • Test — adds or updates tests

What Was Changed [required]

File Change made
utils/recommender.py Added clear_cache() function
tests/test_basic.py Added test for cache invalidation

How to Test This PR

  1. Clone this branch: git checkout main (from CyberHunter12-ui:main)
  2. Install dependencies: pip install -r requirements.txt
  3. Run the app locally: python app.py
  4. Open http://127.0.0.1:5000 and test the new project search fields.
  5. Run the test suite: python tests/test_basic.py

Expected test output:
27 passed, 0 failed out of 27 tests
Running tests/test_basic.py
...........................Ran 27 tests in 0.019sOK (27 passed, 0 failed out of 27 tests)

Self-Review Checklist

  • I have read CONTRIBUTING.md and followed all guidelines
  • My branch name follows the convention: feat/, fix/, docs/, data/, style/, test/
  • I have run python tests/test_basic.py and all 27 tests pass
  • I have run flake8 . locally and there are no errors
  • I have not introduced any print() or console.log() debug statements
  • Every new function I wrote has a docstring
  • I have not modified files outside the scope of the linked issue
  • If I added a project to the dataset, it has all required JSON fields

Notes for Reviewer

The automated GitHub Actions CI environment has already successfully validated the linting (flake8) and test execution across Python 3.9, 3.11, and 3.12 environments.
image

@CyberHunter12-ui
Copy link
Copy Markdown
Author

@komalharshita updated the template

@komalharshita
Copy link
Copy Markdown
Owner

@CyberHunter12-ui resolve the merge conflicts and mention the issue which will close this pr eg. Closes #issue_number

@komalharshita komalharshita added the need review Further information is requested label May 19, 2026
@CyberHunter12-ui
Copy link
Copy Markdown
Author

CyberHunter12-ui commented May 19, 2026

@komalharshita Conflits resolved
Please check

@CyberHunter12-ui
Copy link
Copy Markdown
Author

@komalharshita
I've pushed the fix for the JSON syntax error! Could a maintainer please approve the workflow run so the tests can verify it?

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.

2 participants