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 | Get alumnus projects data #140

Merged
merged 5 commits into from
Jun 8, 2020
Merged

GET | Get alumnus projects data #140

merged 5 commits into from
Jun 8, 2020

Conversation

Mu7ammadAbed
Copy link
Member

  • I made a route to bring alumnus projects, in order to achieve this purpose I wrote a database query which inner join studnt_project table with project table and send the data from project table.

  • I created a function which will handle the route /alumni/:studentId/projects and check the params if every thing is right it will send projects data as a json, if there's something wrong with the inputs a proper response message will be send.

  • After that I do some tests for the function and I achieve 100% test coverage.

Relates #138

…rpose I wrote a database query which inner join studnt_project table with project table and send the data from project table, after that I created a function which will handle the route /alumni/:studentId/projects and check the params if every thing is right it will send projects data as a json, if there's something wrong with the inputs a proper response message will be send.

After that I do some tests for the function and I achieve 100% test coverage
Relates #138
@Mu7ammadAbed Mu7ammadAbed added Awaiting Review This PR is ready and waiting for review Backend priority-1 labels Jun 8, 2020
@Mu7ammadAbed Mu7ammadAbed linked an issue Jun 8, 2020 that may be closed by this pull request
3 tasks
@ranasobeid95 ranasobeid95 merged commit 77d3de2 into master Jun 8, 2020
@ranasobeid95 ranasobeid95 deleted the alumni-projects branch June 8, 2020 16:38
@ranasobeid95 ranasobeid95 added Done and removed Awaiting Review This PR is ready and waiting for review labels Jun 8, 2020
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.

GET | Get Alumnus Projects
3 participants