Skip to content
This repository has been archived by the owner on Dec 30, 2022. It is now read-only.

feat(voiceSearch): add voice search component #668

Merged
merged 19 commits into from
May 21, 2019
Merged

Conversation

eunjae-lee
Copy link
Contributor

@eunjae-lee eunjae-lee commented Apr 29, 2019

Summary

This adds a new voice search component.

Checklist

  • add component
  • add stories
  • add tests
  • replace tmp/ with the ones from published IS.js

@eunjae-lee
Copy link
Contributor Author

I should've made this a draft pr since this cannot be merged at the moment because we need to wait for IS.js to be published.

CHANGELOG.md Outdated Show resolved Hide resolved
src/components/VoiceSearch.vue Outdated Show resolved Hide resolved
src/components/VoiceSearch.vue Outdated Show resolved Hide resolved
Copy link
Contributor

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

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

nice, some more small feedback, but otherwise LGTM

src/components/VoiceSearch.vue Outdated Show resolved Hide resolved
src/components/VoiceSearch.vue Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@Haroenv Haroenv requested review from a team and Haroenv May 14, 2019 12:06
@ghost ghost requested review from tkrugg and removed request for a team May 14, 2019 12:06
Co-Authored-By: Haroen Viaene <fingebimus@me.com>
@eunjae-lee
Copy link
Contributor Author

Let's wait until 3.5.1 is published.

@eunjae-lee eunjae-lee requested a review from Haroenv May 20, 2019 13:48
@eunjae-lee
Copy link
Contributor Author

@Haroenv re-requesting a final review before merge.

@eunjae-lee eunjae-lee merged commit 84efffc into master May 21, 2019
@eunjae-lee eunjae-lee deleted the feat/voice-search branch May 21, 2019 07:56
Haroenv pushed a commit to algolia/instantsearch that referenced this pull request Dec 28, 2022
…rch#668)

* chore(voiceSearch): scope styles to non-ais elements

* feat(voiceSearch): WIP adding voiceSearch component

* test(voiceSearch): add stories

* test(voiceSearch): add tests for the component

* chore(voiceSearch): formatted

* chore(voiceSearch): fix lint errors

* doc: revert a mistake

* chore(voiceSearch): remove unnecessary computed properties

* feat(voiceSearch): provide all data to the default root slot

* feat(voiceSearch): change slot properties to kebab case

* feat(voiceSearch): add a computed property for root slot props

* chore(voiceSearch): replace temp files with the ones from latest IS.js

* fix(voiceSearch): fix an import path

* chore(voiceSearch): update bundlesize

* Apply suggestions from code review

Co-Authored-By: Haroen Viaene <fingebimus@me.com>

* chore: update IS.js@3.5.1

* chore: update bundlesize
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants