Skip to content
This repository has been archived by the owner on Jun 17, 2021. It is now read-only.

Commit

Permalink
add analytics tags to search (#365)
Browse files Browse the repository at this point in the history
* add analytics tags

* Update AddDependencySearchProvider.js

* #352 Disable actions that require an internet connection (#368)

* Online check component added

* add reducer & action for online check component.

also included in App.js

* removed sidebar functionality when offline

* removed depedency functionality when offline

* removed functionality from create new project wizard when offline

* fixed tests to align with disabled pattern

* add infoBanner to Z-indexes for future usage.

* removed random {' '}, changed IS_ONLINE_CHECK to SET_ONLINE_STATUS

* pull request requested changes

missed one file.

* updated infobar to be fixed header.

* flow fix

* remove random whitespace

* Pull request changes

remove state type, move styled components to before redux part, change background color of infobar to be transparent.

* enable menu items based on isOnline

* Setup ESlint rules for Jest (#366)

* WIP: Added eslint-jest & fixed issues (except snapshots)

* reduced snapshot size to meet max 100 lines

* fixed snapshot

* changed comment to a note about noPadding prop

* updated snapshot

* Test ProjectConfigurationModal component (#360)

* WIP: Added render test

* WIP: Added tests

* fixed flow

* added focus test

* Merge branch 'master' into test-project-configuration-modal

* fixed linting warnings

* removed typing as no flow used in test files

* Test CreateNewProjectWizard components (#361)

* WIP: Added tests

* WIP: ProjectType undefined not throwing in test

* fixed flow

* added tests

* reverted to children render props

* Merge branch 'master' into test-create-new-project-wiz

* Fixed linting & replaced snapshot with simple smoke tests

* addressed review comments

* Test DependencyManagementPane component (#363)

* moved related components to DependencyManagementPane

* added tests

* fixed flow

* added a constant instead of Date.now()

* updated snapshot

* removed white-spaces

* removed verbose

* fixed linting & mocked AlgoliaLogo

* move related component files & moved tests to __tests__ folder

* Add script for test:dev

* Fix spaces and update lint-staged (#369)

* Switch to advanced config to silence validation warning

* Fix spacing issues

* chore: fix merge changes
  • Loading branch information
Haroenv authored Mar 27, 2019
1 parent cb6e1f6 commit 4279e23
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ class AddDependencySearchProvider extends Component<Props> {
'humanDownloadsLast30Days',
'license',
]}
analyticsTags={['guppy', 'guppy-search']}
hitsPerPage={10}
/>
{this.props.children}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,7 @@ class DependencyInfoFromNpm extends Component<Props, State> {
<Configure
attributesToRetrieve={['name', 'version', 'modified']}
hitsPerPage={1}
analyticsTags={['guppy', 'guppy-details']}
/>
<FilterByIds attribute="objectID" defaultRefinement={[packageName]} />
<Result packageName={packageName} isOnline={isOnline}>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,12 @@ exports[`AddDependencySearchProvider component should render SearchBox 1`] = `
}
>
<AlgoliaConfigure(UnknownComponent)
analyticsTags={
Array [
"guppy",
"guppy-search",
]
}
attributesToRetrieve={
Array [
"name",
Expand Down

0 comments on commit 4279e23

Please sign in to comment.