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

#161045358 create categories for articles #33

Merged
merged 1 commit into from
Oct 16, 2018

Conversation

zaabu
Copy link

@zaabu zaabu commented Oct 12, 2018

What does this PR do?

  • This PR implements functionality for categorizing articles. This enables authors to assign categories to the articles that they write.

Description of Task to be completed?

Articles should belong to a certain category for example; music, sports, history, art and literature, to mention but a few.

Endpoints

GET /api/categories/ -- Get all categories

GET /api/categories/<cat_name>/ - Get all articles under a speciifc category

How should this be manually tested?

A category is chosen from a predefined choice of categories available. A user should enter category_name as a string. This name must exist in the category choices or else an error will be raised.

Example of an article with a category.

image

What are the relevant pivotal tracker stories?

[Delivers #161045358] (https://www.pivotaltracker.com/story/show/161045358)

@nakiwuge nakiwuge temporarily deployed to ah-backend-lannister-sta-pr-33 October 12, 2018 22:11 Inactive
@zaabu zaabu changed the title #161045358 create article categories #161045358 create categories for articles Oct 12, 2018
@coveralls
Copy link

coveralls commented Oct 13, 2018

Coverage Status

Coverage decreased (-1.0%) to 93.807% when pulling 99ca4a7 on ft-create-article-categories-161045358 into 6d02828 on develop.

@zaabu zaabu force-pushed the ft-create-article-categories-161045358 branch from f20069c to 3958d5e Compare October 14, 2018 15:25
@zaabu zaabu temporarily deployed to ah-backend-lannister-sta-pr-33 October 14, 2018 15:25 Inactive
@zaabu zaabu force-pushed the ft-create-article-categories-161045358 branch from 3958d5e to 3898e39 Compare October 14, 2018 15:26
@zaabu zaabu temporarily deployed to ah-backend-lannister-sta-pr-33 October 14, 2018 15:26 Inactive
@zaabu zaabu force-pushed the ft-create-article-categories-161045358 branch from 3898e39 to 084cdd0 Compare October 15, 2018 18:33
@zaabu zaabu temporarily deployed to ah-backend-lannister-sta-pr-33 October 15, 2018 18:33 Inactive
@zaabu zaabu force-pushed the ft-create-article-categories-161045358 branch from 084cdd0 to 3e2be7a Compare October 16, 2018 05:32
@zaabu zaabu temporarily deployed to ah-backend-lannister-sta-pr-33 October 16, 2018 05:32 Inactive
@zaabu zaabu added the ready PR is ready for merging label Oct 16, 2018
- create category model
- add reference to category model on article

feat(create_categories): setup serializers

- add category name to article serializer
- add category name to article serializer fields

feat(article_categories): set up views category views

- create views for listing all categories
- create views for listing all articles under a certain category
@PatrickCmd PatrickCmd merged commit fd07b04 into develop Oct 16, 2018
@PatrickCmd PatrickCmd deleted the ft-create-article-categories-161045358 branch October 17, 2018 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready PR is ready for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants