Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GET /alumni/projects/:projectId | Route #65

Closed
3 tasks done
ranasobeid95 opened this issue Apr 3, 2020 · 0 comments
Closed
3 tasks done

GET /alumni/projects/:projectId | Route #65

ranasobeid95 opened this issue Apr 3, 2020 · 0 comments
Assignees

Comments

@ranasobeid95
Copy link
Collaborator

ranasobeid95 commented Apr 3, 2020

Route: GET api/v1/alumni/projects/:projectId

Acceptance Criteria

  • DB query

  • Response

  • Testing


DB query
SELECT student.name FROM student INNER JOIN student_project ON student_project.student_id = student.id where project_id = projectId

Response
  • Success
{ StatusCode: 200,
  data: [ Project students ]
}
  • Failed
{statusCode: 404, message: 'Please check project ID you entered ' }
Testing

Successful test if

  • statusCode === 200

#5

@ranasobeid95 ranasobeid95 changed the title GET /alumni/:projectId GET /alumni/projects/:projectId| Route Apr 3, 2020
@ranasobeid95 ranasobeid95 changed the title GET /alumni/projects/:projectId| Route GET /alumni/projects/:projectId | Route Apr 3, 2020
MohammedAlghazali added a commit that referenced this issue Apr 5, 2020
MohammedAlghazali added a commit that referenced this issue Apr 5, 2020
MohammedAlghazali added a commit that referenced this issue Apr 5, 2020
@MohammedAlghazali MohammedAlghazali added Awaiting Review This PR is ready and waiting for review and removed To-Do labels Apr 5, 2020
@ranasobeid95 ranasobeid95 added Done and removed Awaiting Review This PR is ready and waiting for review labels Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants