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.

Sign up for GitHub

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

Jump to bottom

#166240996 - CRUD articles With API Documentation and Tests #24

Merged
merged 1 commit into from
Jun 15, 2019
Merged

#166240996 - CRUD articles With API Documentation and Tests #24

merged 1 commit into from
Jun 15, 2019

Conversation

UhiriweAudace
Copy link
Contributor

@UhiriweAudace UhiriweAudace commented Jun 12, 2019 •
edited
Loading

What does this PR do?

In this PR, I implemented the endpoints that allow a user to be able to create, to get all articles
to get one article based on a slug

Description of Task to be completed?

Having the following endpoints working:
POST /api/articles
GET /api/articles
GET /api/articles/:slug
PUT /api/articles/:slug
DELETE /api/articles/:slug

How should this be manually tested?

  • clone the repository
  • go to your terminal and then write the following command
    cd tesla-ah
  • after that try to install all dependencies that are required for this project, simply by running the following command:
    yarn install or yarn
  • Then running the server, by running the below commands:
    1. sequelize db:migrate for this command make sure that the sequelize-cli package is installed globally
    2. yarn dev
  • for running the tests, write the below command in your terminal
    yarn test

Any background context you want to provide?

N/A

What are the relevant pivotal tracker stories?

#166240996

Screenshots (if appropriate)

N/A

Questions:

@UhiriweAudace UhiriweAudace force-pushed the ft-crud-articles-166240996 branch 2 times, most recently from bdb61ef to dbcee01 Compare June 12, 2019 18:46
@UhiriweAudace UhiriweAudace force-pushed the ft-crud-articles-166240996 branch 5 times, most recently from df49ebe to dc43391 Compare June 14, 2019 08:58
@UhiriweAudace UhiriweAudace force-pushed the ft-crud-articles-166240996 branch 3 times, most recently from 99ccf70 to c3511e7 Compare June 14, 2019 11:34
@UhiriweAudace UhiriweAudace force-pushed the ft-crud-articles-166240996 branch 4 times, most recently from 8358601 to efbfad1 Compare June 15, 2019 11:44
@UhiriweAudace UhiriweAudace force-pushed the ft-crud-articles-166240996 branch 2 times, most recently from e3e5352 to d00d685 Compare June 15, 2019 13:41
@Quantum-35 Quantum-35 merged commit 1e59ae0 into develop Jun 15, 2019
@UhiriweAudace UhiriweAudace added Merged this is for the merged branch and removed Needs Review labels Jun 17, 2019
Diama1 pushed a commit that referenced this pull request Jul 17, 2019
@UhiriweAudace @Diama1
[ feature #166240996] -CRUD articles With API Documentation and Tests (…
3dbf5cb 
…#24)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eliemugenzi eliemugenzi eliemugenzi approved these changes

@Quantum-35 Quantum-35 Quantum-35 approved these changes

@rukundoeric rukundoeric rukundoeric approved these changes

@mifeille mifeille Awaiting requested review from mifeille

@deschantkn deschantkn Awaiting requested review from deschantkn

@Diama1 Diama1 Awaiting requested review from Diama1

Assignees
No one assigned
Labels
Merged this is for the merged branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@UhiriweAudace @eliemugenzi @Quantum-35 @rukundoeric