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

fix(ua): Update the User-Agent to use the new format #3616

Merged
merged 4 commits into from
Mar 22, 2019

Conversation

robertmogos
Copy link
Contributor

Summary
Update the UA from instantsearch.js ${version} to instantsearch.js (${version})

@algobot
Copy link
Contributor

algobot commented Mar 22, 2019

Deploy preview for instantsearchjs ready!

Built with commit 9f2ce92

https://deploy-preview-3616--instantsearchjs.netlify.com

@francoischalifour
Copy link
Member

Can we add a test for this?

An appropriate place could be InstantSearch-test.js.

@robertmogos
Copy link
Contributor Author

Can we add a test for this?

An appropriate place could be InstantSearch-test.js.

Yes, it is on my todo list. I might need some help with that

@francoischalifour
Copy link
Member

francoischalifour commented Mar 22, 2019

You can see in the test that we mocked the JavaScript client, with only a search method for now.

The official JavaScript client also exposes an addAlgoliaAgent method. You can therefore add this method to the client to mock the addAlgoliaAgent function with jest.fn(). Then, you're able to check that it has been called correctly at instantiation.

Let me know if you need further help.

Copy link
Contributor

@samouss samouss left a comment

Choose a reason for hiding this comment

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

The build is failing: run yarn lint:fix should fix the issue.

@robertmogos robertmogos merged commit ab84c57 into develop Mar 22, 2019
@samouss samouss deleted the fix/user-agent branch March 22, 2019 13:37
francoischalifour added a commit that referenced this pull request Apr 11, 2019
# [3.3.0](v3.2.1...v3.3.0) (2019-04-11)

### Bug Fixes

* **connectQueryRules:** improve tracked refinement type ([#3648](#3648)) ([e16ad57](e16ad57))
* **currentRefinements:** don't rely on  ([#3672](#3672)) ([cd64bcf](cd64bcf))
* **queryRuleCustomData:** add default template ([#3650](#3650)) ([83e9eaa](83e9eaa))
* **QueryRuleCustomData:** pass data as object to templates ([#3647](#3647)) ([b8f8b4e](b8f8b4e))
* **queryRules:** fix types and stories ([#3670](#3670)) ([ba6e2e6](ba6e2e6))
* **routing:** apply windowTitle on first load ([#3669](#3669)) ([d553502](d553502)), closes [#3667](#3667)
* **routing:** support parsing URLs with up to 100 refinements ([#3671](#3671)) ([6ddcfb6](6ddcfb6))
* **RoutingManager:** avoid stale uiState ([#3630](#3630)) ([e1588aa](e1588aa))
* **types:** improve InstantSearch types ([#3651](#3651)) ([db9b91e](db9b91e))
* **ua:** Update the User-Agent to use the new format ([#3616](#3616)) ([ab84c57](ab84c57))

### Features

* **infiniteHits:** add previous button ([#3645](#3645)) ([2c9e38d](2c9e38d))
* **queryRules:** add connectQueryRules connector ([#3597](#3597)) ([924cd99](924cd99)), closes [#3599](#3599) [#3600](#3600)
* **queryRules:** add context features to Query Rules ([#3617](#3617)) ([922879e](922879e)), closes [#3602](#3602)

### Reverts

* feat(infiniteHits): add previous button ([214c0fc](214c0fc))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants