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

#161217628 User can search instantly on the homepage #41

Merged
merged 1 commit into from
Oct 18, 2018

Conversation

ascii-dev
Copy link
Contributor

What does this PR do?

 - Allow a user get search result without having to press enter

Description of Task to be completed?

 - search result is retrieved as soon as users type

How should this be manually tested?

 - git pull
 - npm i
 - npm run dev
 - login to the application
 - type something into the search bar on the homepage

Any background context you want to provide?

Before now, on the homepage, when a user searches for a team, they have to press the enter key before the search is been made, but now, once they start typing, the search is made.

What are the relevant pivotal tracker stories?

#161217628

Screenshots or gifs (if appropriate)

   N/A

Questions:

   N/A

@nouwatinjacob nouwatinjacob temporarily deployed to ghoulies-taps-client-pr-41 October 16, 2018 09:06 Inactive
Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react'
Referenced from: .eslintrc
Error: Cannot find module 'eslint-config-react'
    at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19)
    at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25)
    at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26)
    at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36)
    at Array.reduceRight ()
    at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28)
    at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22)
    at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20)
    at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44)
    at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)

@ascii-dev ascii-dev force-pushed the ft-user-get-instant-result-161217628 branch from 651191c to e6fbf93 Compare October 16, 2018 15:36
@nouwatinjacob nouwatinjacob temporarily deployed to ghoulies-taps-client-pr-41 October 16, 2018 15:36 Inactive
Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react'
Referenced from: .eslintrc
Error: Cannot find module 'eslint-config-react'
    at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19)
    at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25)
    at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26)
    at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36)
    at Array.reduceRight ()
    at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28)
    at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22)
    at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20)
    at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44)
    at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)

@coveralls
Copy link

coveralls commented Oct 17, 2018

Pull Request Test Coverage Report for Build 383

  • 1 of 7 (14.29%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.4%) to 49.762%

Changes Missing Coverage Covered Lines Changed/Added Lines %
client/src/modules/common/Navbar/index.jsx 0 6 0.0%
Totals Coverage Status
Change from base Build 381: -0.4%
Covered Lines: 388
Relevant Lines: 743

💛 - Coveralls

 - search result is retrieved as soon as users type
[Finishes #161217628]
@ascii-dev ascii-dev force-pushed the ft-user-get-instant-result-161217628 branch from e6fbf93 to 68870ee Compare October 18, 2018 15:32
Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

Cannot find module 'eslint-config-react'
Cannot find module 'eslint-config-react'
Referenced from: .eslintrc
Error: Cannot find module 'eslint-config-react'
    at ModuleResolver.resolve (/home/linters/node_modules/eslint/lib/util/module-resolver.js:74:19)
    at resolve (/home/linters/node_modules/eslint/lib/config/config-file.js:471:25)
    at load (/home/linters/node_modules/eslint/lib/config/config-file.js:542:26)
    at configExtends.reduceRight (/home/linters/node_modules/eslint/lib/config/config-file.js:421:36)
    at Array.reduceRight ()
    at applyExtends (/home/linters/node_modules/eslint/lib/config/config-file.js:403:28)
    at loadFromDisk (/home/linters/node_modules/eslint/lib/config/config-file.js:514:22)
    at Object.load (/home/linters/node_modules/eslint/lib/config/config-file.js:550:20)
    at Config.getLocalConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:228:44)
    at Config.getConfigHierarchy (/home/linters/node_modules/eslint/lib/config.js:180:43)

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.

None yet

5 participants