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

166573650 Fix get all exercises #15

Closed
wants to merge 1 commit into from

Conversation

wekesa931
Copy link
Contributor

@wekesa931 wekesa931 commented Jun 12, 2019

Fix the fetch all exercise endpoint which was not working as expected

Steps to reproduce

After installation and the app runs, navigate to the endpoint: http://127.0.0.1:8000/api/v2/exercise/ and make a get request.

Expected

All exercises should be displayed.

Actual

The exercises are not displayed as expected.

@wekesa931 wekesa931 force-pushed the bg-#166573650-fix-fetch-exercises branch from d740503 to 85f8d59 Compare June 12, 2019 04:46
- Fix the fetch all exercise endpoint which was not working as expected
[Fixes #166573650]
Copy link
Contributor

@jmusila jmusila left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@wangonya wangonya left a comment

Choose a reason for hiding this comment

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

Kindly fix your PR description.

  • Steps to reproduce should be what you have to do to experience the bug (e.g access a certain endpoint)
  • Expected should be how the app should be working ideally (e.g a list of all exercises should be returned)
  • Actual should be what is actually happening as a result of the bug (e.g no exercises are being returned)

Copy link
Contributor

@willeswa willeswa left a comment

Choose a reason for hiding this comment

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

Could you consider naming the branch to follow the Andela conventions?!

@ceexon
Copy link
Contributor

ceexon commented Jun 12, 2019

LGTM

@wekesa931 wekesa931 closed this Jun 12, 2019
@wekesa931 wekesa931 deleted the bg-#166573650-fix-fetch-exercises branch June 12, 2019 13:04
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.

5 participants