Skip to content

Conversation

@denniwang
Copy link
Contributor

ℹ️ Issue

Closes #60

📝 Description

Created endpoint at /projects/{project_id} to fetch information about a project

✔️ Verification

  • wrote jest tests

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.

The put endpoint actually doesn't need an ID, just the project information. ID is created automatically

nourshoreibah
nourshoreibah previously approved these changes Nov 19, 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.

oops wrong pr this is all good!

Rayna-Yu
Rayna-Yu previously approved these changes Nov 20, 2025
Copy link
Contributor

@Rayna-Yu Rayna-Yu left a comment

Choose a reason for hiding this comment

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

Looks good!

@denniwang denniwang dismissed stale reviews from Rayna-Yu and nourshoreibah via 34a0fbb November 21, 2025 02:46
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.

Good job with merge conflicts! we'd want to separate the unit tests vs e2e more explicitly but this was pre that spike so all good to merge

nourshoreibah
nourshoreibah previously approved these changes Nov 21, 2025
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 Get Project By ID Route (GET /projects/{id})

4 participants