Skip to content

feat: personalise <title> tag on project detail page with level and skill#156

Open
Harshada-Yele wants to merge 2 commits into
komalharshita:mainfrom
Harshada-Yele:feat/personalise-project-detail-title
Open

feat: personalise <title> tag on project detail page with level and skill#156
Harshada-Yele wants to merge 2 commits into
komalharshita:mainfrom
Harshada-Yele:feat/personalise-project-detail-title

Conversation

@Harshada-Yele
Copy link
Copy Markdown
Contributor

@Harshada-Yele Harshada-Yele commented May 16, 2026

Summary

The project detail page <title> tag previously rendered as
Project Name — DevPath, which is functional but misses an SEO
opportunity. This PR updates the title in templates/project.html
to include the project's difficulty level and primary skill/interest
area, making pages more discoverable for users searching for specific
project ideas (e.g. "Beginner Python projects"). A Jinja2 {% if %}
guard is included to handle projects with an empty skills list safely.

Related Issue

Closes #84

Type of Change

  • 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

File Change made
templates/project.html Updated <title> tag to include project.level and project.skills[0] with an {% if project.skills %} guard

How to Test This PR

  1. Clone this branch: git checkout feat/personalise-project-detail-title
  2. Install dependencies: pip install -r requirements.txt
  3. Run the app: python app.py
  4. Open http://127.0.0.1:5000 and navigate to any project detail page
  5. Check the browser tab — title should now read:
    Personal Expense Tracker — Beginner Python Project | DevPath
    instead of Personal Expense Tracker — DevPath
  6. Run the tests: python tests/test_basic.py

Expected test output:
27 passed, 0 failed out of 27 tests

Test Results

image
Before After

|
image
|
image
|

Self-Review Checklist

  • I have read CONTRIBUTING.md and followed all guidelines
  • My branch name follows the convention: feat/personalise-project-detail-title
  • 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 ← N/A

Notes for Reviewer

The only file changed is templates/project.html — one line. No Python
logic, no routes, no data files were touched. The {% if project.skills %}
guard ensures the title renders gracefully even if a project entry has an
empty or missing skills list.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 16, 2026

@Harshada-Yele 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.

@Harshada-Yele
Copy link
Copy Markdown
Contributor Author

Hi @komalharshita
Ready for review.
All 27 tests pass and before/after screenshots are included.

@komalharshita
Copy link
Copy Markdown
Owner

@Harshada-Yele kindly resolve the merge conflicts

@komalharshita komalharshita added the need review Further information is requested label May 17, 2026
@Harshada-Yele
Copy link
Copy Markdown
Contributor Author

Hi @komalharshita
Merge conflicts resolved.
Ready for review.

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.

Add <title> tag personalisation to project detail page using project interest area

2 participants