Skip to content

/ vue-apollo

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

Jump to bottom

(V4) Update @vue/composition-api to v0.6.1 #1000

Merged
merged 4 commits into from Jul 27, 2020
Merged

(V4) Update @vue/composition-api to v0.6.1 #1000

merged 4 commits into from Jul 27, 2020

Conversation

@mugi-uno
Copy link

@mugi-uno mugi-uno commented Jun 10, 2020 •
edited

this fixes #1001

With the change of @vue/composition-api v0.6.0, the behavior of the watch API changed.

https://github.com/vuejs/composition-api/ releases/tag/v0.6.0

The lazy option of watch has been replaced by the The opposite immediate option, which defaults to false. (It's ignored when using the effect signature).

This will prevent useQuery from executing the query immediately.

This PR includes the following changes

  • Update @vue/composition-api (to v0.6.1)
  • Add immediate option to the use of the watch API
  • Fix type errors
mugi-uno added 3 commits Jun 10, 2020
@mugi-uno
chore: update deps
5afd149
@mugi-uno
fix(useMutation,useSubscription): use "immediate" option
8714604
@mugi-uno
fix(type): paramToRef and paramToReactive
Loading status checks…
0f48bf7
@liaoliaojun
Copy link

@liaoliaojun liaoliaojun commented Jun 30, 2020

@Akryum Can I help you
@Qubeo Qubeo mentioned this pull request Jul 3, 2020
Closed
@lewebsimple
Copy link

@lewebsimple lewebsimple commented Jul 6, 2020

Any update on this ?
@DarkLite1
Copy link

@DarkLite1 DarkLite1 commented Jul 14, 2020 •
edited

I've been searching for ages about this. Even on my StackOverflow question no one responded. Thanks @Akryum for adding this fix so we can work with the latest @vue/composition-api
@Loremaster
Copy link

@Loremaster Loremaster commented Jul 15, 2020

@Akryum is there anything holding you from merging this PR?
@urbantrout
Copy link

@urbantrout urbantrout commented Jul 23, 2020

@Akryum Would love to see this one merged…
@kyosifov
Copy link

@kyosifov kyosifov commented Jul 24, 2020

It's been a month since updates on this project from @Akryum. Isn't there another maintainer for this project?

@Akryum there are people willing to help with this project.
@Akryum
fix: remove unused import
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
5ba58a8
@Akryum Akryum merged commit e8e8f54 into vuejs:v4 Jul 27, 2020
6 checks passed
6 checks passed
Header rules No header rules processed
Details
Pages changed 77 new files uploaded
Details
Redirect rules No redirect rules processed
Details
Mixed content No mixed content detected
Details
ci/circleci: build Your tests passed on CircleCI!
Details
deploy/netlify Deploy preview ready!
Details
@mugi-uno mugi-uno deleted the mugi-uno:v4 branch Jul 28, 2020
@brunomperes brunomperes mentioned this pull request Sep 3, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

8 participants
@mugi-uno @liaoliaojun @lewebsimple @DarkLite1 @Loremaster @urbantrout @kyosifov @Akryum
You can’t perform that action at this time.