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

#166241007 - Search Functionality Added to Authors Haven #53

Merged
merged 1 commit into from
Jun 28, 2019

Conversation

UhiriweAudace
Copy link
Contributor

@UhiriweAudace UhiriweAudace commented Jun 24, 2019

What does this PR do?

In this PR, I implemented the endpoints that will enable any user/client to send the request for making a search in Authors Haven

Description of Task to be completed?

Having the following Endpoints working:
GET /api/articles?author=<Author's Username>
GET /api/articles?tag=<Article's tag>
GET /api/articles?title=<Article's title>
GET /api/articles?keywords=<Keywords>

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
    3. After running the server, Open your postman, and then try to send for searching the article based on the provided tag, author's name, article's title or any keywords
  • for running the tests, write the below command in your terminal
    yarn test:local

Any background context you want to provide?

N/A

What are the relevant pivotal tracker stories?

#166241007

Screenshots (if appropriate)

N/A

Questions:

Before You Code
https://docs.google.com/document/d/1Dh6VZBn61Y2b-lvV8oOafSoH1m7ilT0wXf6iDXAcz8o/edit?usp=sharing

@UhiriweAudace UhiriweAudace force-pushed the ft-search-functionality-166241007 branch 7 times, most recently from 5cac4c8 to 70eb0af Compare June 26, 2019 12:10
@UhiriweAudace UhiriweAudace force-pushed the ft-search-functionality-166241007 branch from 70eb0af to 425bfc0 Compare June 26, 2019 19:41
@UhiriweAudace UhiriweAudace force-pushed the ft-search-functionality-166241007 branch from 425bfc0 to 7ab121f Compare June 27, 2019 08:56
@UhiriweAudace UhiriweAudace added Needs Merge Needs To merge and removed Needs Review labels Jun 27, 2019
@UhiriweAudace UhiriweAudace force-pushed the ft-search-functionality-166241007 branch 12 times, most recently from 183646e to a5d1966 Compare June 28, 2019 15:32
.get('/api/articles?page=1&limit=10')
.send((err, res) => {
console.log('=======FOUND RESULT======');
console.log(res.body);
Copy link

Choose a reason for hiding this comment

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

Unexpected console statement no-console

.request(app)
.get('/api/articles?page=1&limit=10')
.send((err, res) => {
console.log('=======FOUND RESULT======');
Copy link

Choose a reason for hiding this comment

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

Unexpected console statement no-console

@UhiriweAudace UhiriweAudace force-pushed the ft-search-functionality-166241007 branch from a5d1966 to 03ff96d Compare June 28, 2019 15:36
@Quantum-35 Quantum-35 merged commit 28c0010 into develop Jun 28, 2019
@UhiriweAudace UhiriweAudace added Merged this is for the merged branch and removed Needs Merge Needs To merge labels Jul 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged this is for the merged branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants