Skip to content

feat: add dynamic fallback message for empty area of interest#297

Open
HariniGS-1207 wants to merge 2 commits into
komalharshita:mainfrom
HariniGS-1207:fix/add-more-intrest-section
Open

feat: add dynamic fallback message for empty area of interest#297
HariniGS-1207 wants to merge 2 commits into
komalharshita:mainfrom
HariniGS-1207:fix/add-more-intrest-section

Conversation

@HariniGS-1207
Copy link
Copy Markdown

@HariniGS-1207 HariniGS-1207 commented May 18, 2026

Summary
This PR adds a dynamic fallback message for empty areas of interest. When a user selects a technical interest category that does not currently have any project recommendations available in the repository, the application will now return an empty projects array along with a user-friendly message rather than breaking or throwing a generic error.
Related Issue [required]

Closes #297 Type of Change

  • Feature — adds new functionality.
  • Style — CSS or visual changes only, no logic change.

How to Test This PR

  • Clone this branch: git checkout fix/add-more-intrest-section.
  • Install dependencies: pip install -r requirements.txt.
  • Run the app: python app.py
  • Open http://127.0.0.1:5000/ and select one of the newly added interest dropdown areas (e.g., "Machine Learning/AI").
  • Verify that the UI gracefully hides the grid and displays: "No projects are currently available for this interest. Please check back later or try a different area."

Run the tests: python tests/test_basic.py

**Test Results **
28 passed, 0 failed out of 28 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 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).

ScreenShots

Screenshot 2026-05-18 211107 Screenshot 2026-05-18 211053

Notes for Reviewer
None.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

@HariniGS-1207 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.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Thank you for submitting your first pull request to DevPath.

Before review:

  • Complete the PR template fully
  • Ensure all tests pass
  • Link your PR to an issue
  • Keep changes scoped to the issue

A maintainer will review your contribution soon.

@HariniGS-1207
Copy link
Copy Markdown
Author

Hi @komalharshita ,

I have completed the PR template, linked the issue, and verified that all 28 tests pass successfully locally.
Could a you please approve the workflows to run and review my changes when you have a moment? Thank you!

@komalharshita
Copy link
Copy Markdown
Owner

@HariniGS-1207 ensure that all checks pass before merging

@HariniGS-1207
Copy link
Copy Markdown
Author

Hi @komalharshita, the Vercel check is currently waiting for authorization from a maintainer to run. Could you please approve/authorize the workflow so the check can pass? All local tests are passing successfully!

@komalharshita
Copy link
Copy Markdown
Owner

@HariniGS-1207 resolve the merge conflicts

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