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

feat(vue 3): support vue 3 #990

Merged
merged 45 commits into from
Aug 23, 2021
Merged

feat(vue 3): support vue 3 #990

merged 45 commits into from
Aug 23, 2021

Conversation

eunjae-lee
Copy link
Contributor

@eunjae-lee eunjae-lee commented Jun 1, 2021

Summary

This is a base branch for all the Vue 3 related PRs.

How to try this branch

  1. scroll to bottom
  2. click the status icon of the latest commit
  3. click "Details" on "ci/codesandbox" item
  4. get the url to install the temporary version from this branch
  5. import like this:
import InstantSearch from 'vue-instantsearch/vue3/es'
  1. let us know how it goes and if you see any bug
CleanShot.2021-07-02.at.09.59.52.mp4

Major TODOs

  • vue-router@4: It just works, only with one change on the userland: vueRouter.currentRoute.queryvueRouter.currentRoute.value.query (need update the doc and the example)
  • Vue 3 with vue-cli
  • Vue 3 with vite
  • Vue 3 SSR with vue-cli
  • Vue 3 SSR with vite
  • Nuxt 3 (when it becomes public beta, which isn't yet)

Haroenv and others added 7 commits June 1, 2021 14:38
Co-authored-by: eunjae-lee <karis612@gmail.com>
v-model used to be @input / :value, but is now @update:modelValue / :modelValue. Since those don't overlap, it's possible to support both, by using them as fallback

"emits" is now also required in v3, so I added this to all components emitting

docs: https://v3.vuejs.org/guide/migration/v-model.html
Co-authored-by: eunjae-lee <karis612@gmail.com>
Co-authored-by: eunjae-lee <karis612@gmail.com>
* feat(vue 3): support both defineComponent and component

* chore: clean up with vue-demi
@eunjae-lee eunjae-lee added this to the Vue 3 milestone Jun 1, 2021
@codesandbox-ci
Copy link

codesandbox-ci bot commented Jun 1, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit dcdf81c:

Sandbox Source
vue-instantsearch-e-commerce Configuration

@Kocal
Copy link
Contributor

Kocal commented Jun 1, 2021

Nice job @eunjae-lee! ✨ ❤️
image

@Haroenv Haroenv mentioned this pull request Jun 17, 2021
@Haroenv
Copy link
Contributor

Haroenv commented Aug 17, 2021

let's make sure we rebase instead of squashing this PR to preserve the history and changelog

@eunjae-lee eunjae-lee marked this pull request as ready for review August 17, 2021 12:55
Eunjae Lee and others added 7 commits August 19, 2021 11:08
* chore(v3 compat): shorten import path for vue3

* chore: change bundle output directory for vue 2

* chore: add dist/ back to .gitignore

* chore: add dist/ back to .eslintignore

* reorden gitignore

* Update package.json

Co-authored-by: Haroen Viaene <hello@haroen.me>

Co-authored-by: Haroen Viaene <hello@haroen.me>
* chore: release v4.0.0-beta.2

* Update CHANGELOG.md

Co-authored-by: eunjae-lee <karis612@gmail.com>
Co-authored-by: Eunjae Lee <eunjae.lee@algolia.com>
…ServerRootMixin (#1044)

* fix(vue 3): pass renderToString to findResultsState instead of createServerRootMixin

* fix: change findResultsState to accept an object as props
Co-authored-by: eunjae-lee <karis612@gmail.com>
@@ -1,3 +1,60 @@
# [4.0.0-beta.3](https://github.com/algolia/vue-instantsearch/compare/v4.0.0-beta.2...v4.0.0-beta.3) (2021-08-20)
Copy link
Contributor

Choose a reason for hiding this comment

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

should we change this to just say v4 and have all changes to v3?

Copy link
Contributor Author

@eunjae-lee eunjae-lee Aug 23, 2021

Choose a reason for hiding this comment

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

Yes, I'm planning to update it in the release PR for 4.0.0.

@eunjae-lee eunjae-lee merged commit f02a173 into master Aug 23, 2021
@eunjae-lee eunjae-lee deleted the feat/vue3-compat branch August 23, 2021 12:54
Haroenv pushed a commit to algolia/instantsearch that referenced this pull request Dec 28, 2022
…feat/vue3-compat

feat(vue 3): support vue 3
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.

5 participants