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

[Bug] Fix typos exercise all information endpoint #17

Merged
merged 1 commit into from
Aug 13, 2018

Conversation

wasswa-derick
Copy link
Contributor

What does this PR do?

  • Fixes a typo on exercise all information endpoint.

Description of Task to be completed?

  • The endpoint title has a typo and this is replaced with a correct one.
  • The breadcrumbs on the endpoint have a typo as well and this is corrected.

How should this be manually tested?

  • Clone the project, https://github.com/andela/wg-wits.git.
  • In your terminal, cd wg-wits
  • Checkout to the bg-fix-exercises-endpoint-typo-159236197 branch.
  • In the project directory, create a virtual environment and activate it: run virtualenv -p python3 venv-django then source venv-django/bin/activate
  • Install requirements pip install -r requirements_devel.txt
  • Serve the application python manage.py runserver
  • In the browser, visit the endpoint for a particular exercise and show its information with, http://127.0.0.1:8000/api/v2/exercise/<exerciseID>/info/
  • The exercise information endpoint and the breadcrumb should look like the ones in the image below.

What are the relevant pivotal tracker stories?

#159236197

Screenshots (if appropriate)

Title and breadcrumbs for the exercise information endpoint.
screen shot 2018-08-10 at 12 51 40

@coveralls
Copy link

Pull Request Test Coverage Report for Build 308

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 93.217%

Totals Coverage Status
Change from base Build 298: 0.0%
Covered Lines: 12300
Relevant Lines: 13195

💛 - Coveralls

@lym lym merged commit c5ccc6f into develop Aug 13, 2018
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.

3 participants