Skip to content

Conversation

@Ruchitha12
Copy link
Member

Fixes #72

What changes did you make?

  • Added new soc_broad model
  • Implemented new API endpoint
  • Added unit tests for the model and API

Why did you make the changes (we will use this info to test)?

Screenshots of Proposed Changes to the Website (if any, please do not screenshot code changes)

No UI - Backend changes only

@Ruchitha12 Ruchitha12 changed the title Create Table : soc_broad | Model | API | Test cases Create Table - soc_broad | Model | API | Test cases Dec 3, 2025
Copy link
Member

@fyliu fyliu left a comment

Choose a reason for hiding this comment

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

Looks great. Thanks!

This one is missing the initial data, but the model looks good.

@github-project-automation github-project-automation bot moved this to PR changes requested in P: PD: Project Board Jan 15, 2026
@Ruchitha12
Copy link
Member Author

@fyliu Can you please confirm if this is the data that needs to be added?

https://docs.google.com/spreadsheets/d/1x_zZ8JLS2hO-zG0jUocOJmX16jh-DF5dccrd_OEGNZ0/edit?gid=1319402959#gid=1319402959

@Ruchitha12
Copy link
Member Author

@fyliu Added the code to add initial data into soc_broad table

@Ruchitha12 Ruchitha12 requested a review from fyliu January 20, 2026 20:00
@fyliu fyliu moved this from PR changes requested to 👀PR being reviewed in P: PD: Project Board Jan 20, 2026
Copy link
Member

@fyliu fyliu left a comment

Choose a reason for hiding this comment

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

Looks great. Thanks!

@fyliu fyliu force-pushed the feature/soc_broad branch from 8eca74c to 0cd010d Compare January 20, 2026 23:40
@fyliu
Copy link
Member

fyliu commented Jan 20, 2026

I merged the last pre-commit formatting commit into the previous one.

@fyliu fyliu merged commit ce6b78a into hackforla:main Jan 20, 2026
2 checks passed
@github-project-automation github-project-automation bot moved this from 👀PR being reviewed to ✅Done in P: PD: Project Board Jan 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅Done

Development

Successfully merging this pull request may close these issues.

Create Table: soc_broad

2 participants