Skip to content

Conversation

@Yurika-Kan
Copy link
Contributor

@Yurika-Kan Yurika-Kan commented Oct 30, 2025

ℹ️ Issue

Closes #54

📝 Description

User can post a project, by giving at least a name!

Briefly list the changes made to the code:

  1. Added POST /projects endpoint through Lambda CLI
  2. Configured local db setup for testing
  3. Implemented logic for POST - all fields optional except for name, light validation logic if fields were present
  4. Tested with unit + end to end tests and Swagger UI and checking local database
  5. Tested with curling while server was running

✔️ Verification

Unit tests & end to end tests:

  • tested minimum & maximum fields case
  • tested invalid & valid fields case
  • tested with server running & curling:
    • cd into apps/backend/lambdas/projects -> npm run dev
    • POST with name + budget: 201 Created
    • POST with only name: 201 Created
    • POST without name: 400 Bad Request (validation works)
    • Swagger JSON: shows name as required

Swagger UI:
Screenshot 2025-10-30 at 5 06 20 PM
Screenshot 2025-10-30 at 4 34 53 PM

Postgres (entries from test):
Screenshot 2025-10-30 at 5 22 00 PM

🏕️ (Optional) Future Work / Notes

Standardizing testing configurations/set up especially with Jest & TS
Note: end to end test must be run with concurrent terminal running npm run dev

nourshoreibah
nourshoreibah previously approved these changes Nov 6, 2025
Copy link
Collaborator

@nourshoreibah nourshoreibah left a comment

Choose a reason for hiding this comment

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

Great work!!

denniwang
denniwang previously approved these changes Nov 6, 2025
Copy link
Contributor

@denniwang denniwang left a comment

Choose a reason for hiding this comment

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

i approve this pr

@Yurika-Kan Yurika-Kan dismissed stale reviews from denniwang and nourshoreibah via 63e12cf November 6, 2025 22:18
Copy link
Collaborator

@nourshoreibah nourshoreibah left a comment

Choose a reason for hiding this comment

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

Reapproving!

@Yurika-Kan Yurika-Kan requested a review from denniwang November 29, 2025 02:20
Copy link
Contributor

@denniwang denniwang left a comment

Choose a reason for hiding this comment

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

i approve of this pull request

@Yurika-Kan Yurika-Kan merged commit 37db9c1 into main Nov 29, 2025
4 checks passed
@Yurika-Kan Yurika-Kan deleted the 54-create-project-route branch November 29, 2025 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Projects] Add Create Project Route (POST /projects)

4 participants