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

feat(vue): add vue3 + vite template #558

Merged
merged 3 commits into from
Feb 11, 2022
Merged

feat(vue): add vue3 + vite template #558

merged 3 commits into from
Feb 11, 2022

Conversation

dhayab
Copy link
Member

@dhayab dhayab commented Feb 10, 2022

To help with an upcoming guide on integrating autocomplete within a vue-instantsearch app, I thought it would be interesting to have a ready to use boilerplate for a vue instantsearch with vue 3.

I opted to use Vite instead of vue cli, but I can switch back if there is an issue I am not aware of.

@dhayab dhayab marked this pull request as draft February 10, 2022 17:36
@dhayab dhayab marked this pull request as ready for review February 11, 2022 09:13
@dhayab dhayab requested review from a team, FabienMotte and Haroenv and removed request for a team February 11, 2022 09:13
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.

👍 code looks impeccable, I'd change the name

const install = require('../../tasks/node/install');
const teardown = require('../../tasks/node/teardown');

module.exports = {
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe instead of "with vite", it should lead with "with vue 3", what do you think?

Copy link
Member Author

Choose a reason for hiding this comment

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

It's okay with me, I was not sure how opinionated we could be by chosing Vite over other tools.

Copy link
Contributor

Choose a reason for hiding this comment

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

We're opinionated in the tools for the other examples too, so I don't think this really has an impact. As long as it works on codesandbox easily, any tool is fine

@dhayab dhayab merged commit 117067e into master Feb 11, 2022
@dhayab dhayab deleted the feat/vue3-vite-template branch February 11, 2022 10:20
aymeric-giraudet pushed a commit to algolia/instantsearch that referenced this pull request Dec 14, 2022
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