Skip to content

data: add three new beginner-level projects to dataset#72

Merged
komalharshita merged 2 commits into
komalharshita:mainfrom
Sagun-Bajpai:data/add-three-beginner-projects
May 24, 2026
Merged

data: add three new beginner-level projects to dataset#72
komalharshita merged 2 commits into
komalharshita:mainfrom
Sagun-Bajpai:data/add-three-beginner-projects

Conversation

@Sagun-Bajpai

Copy link
Copy Markdown
Contributor

Summary [required]

This PR adds three new beginner-level projects to the dataset to give users more project options, especially in underrepresented interest areas like Games and Automation. Three new entries have been added to data/projects.json along with their starter code templates.

Related Issue [required]

Closes #4

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
data/projects.json Added 3 new beginner projects (IDs 8, 9, 10)
starter_code/number_guessing.py Added starter template for Number Guessing Game
starter_code/email_automation.py Added starter template for Simple Email Automation
starter_code/quiz_app.html Added starter template for Quiz App

How to Test This PR [required]

  1. Clone this branch: git checkout data/add-three-beginner-projects
  2. Install dependencies: pip install -r requirements.txt
  3. Run the app: python app.py
  4. Open http://127.0.0.1:5000 and select Games or Automation interest
  5. Run the tests: python tests/test_basic.py

Expected test output:

Signed-off-by: sagun bajpai <sagunbajpaisagun@gmail.com>
@vercel

vercel Bot commented May 15, 2026

Copy link
Copy Markdown

@Sagun-Bajpai 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.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

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.

@komalharshita komalharshita added need review Further information is requested level:beginner quality:clean labels May 17, 2026

@komalharshita komalharshita left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thank you for the contribution — this is a well-structured dataset expansion overall. I appreciate that you not only added new project entries, but also included matching starter templates and kept the JSON schema consistent across all projects.

The Games and Automation categories are also a good addition since they improve project diversity for beginners.

Before merge, there are two improvements needed:

  1. In starter_code/email_automation.py, the example usage currently includes:
send_email("your@gmail.com", "yourpassword", ...)

Even though these are placeholders, it encourages hardcoded credential patterns for beginners. Please replace this with a safer example approach (environment variables, input prompts, or commented placeholders).

  1. starter_code/quiz_app.html is currently very minimal and mostly TODO comments. Please add a slightly stronger starter scaffold (basic buttons/question structure/score section) so contributors have a more meaningful starting point.

Once these are updated, this PR will be in a strong state for merge.

Signed-off-by: sagun bajpai <sagunbajpaisagun@gmail.com>
@Sagun-Bajpai Sagun-Bajpai force-pushed the data/add-three-beginner-projects branch from 4b0da12 to cece343 Compare May 20, 2026 03:17
@Sagun-Bajpai

Copy link
Copy Markdown
Contributor Author

Hi @komalharshita 👋

I have made both the requested changes:

  1. ✅ email_automation.py — replaced hardcoded password with os.environ and input() prompt
  2. ✅ quiz_app.html — added proper scaffold with buttons, score display, and result screen

All 27 tests passing. Please review when you get a chance. Thank you! 😊

@komalharshita komalharshita left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for addressing the earlier review feedback. The PR is now in a much stronger state.

The dataset additions are well-structured, educationally useful, and improve beginner project diversity in the Games and Automation categories. I also appreciate that the matching starter templates were included and kept consistent with the existing schema/style of the repository.

The previously raised concerns were handled well:

  • the email automation example now avoids insecure hardcoded credential patterns
  • the quiz app starter scaffold is now much more meaningful for beginners

The implementation remains properly scoped to dataset/template additions without introducing unrelated modifications, which keeps maintenance risk low.

Overall, this is now a solid contribution and merge-ready.

Approved for merge.

@komalharshita komalharshita merged commit 41daf62 into komalharshita:main May 24, 2026
7 of 8 checks passed
@komalharshita komalharshita added type:docs Improvements or additions to documentation type:feature gssoc:approved and removed need review Further information is requested labels May 24, 2026
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 three new beginner-level projects to the dataset

2 participants