Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

fix: Query names in refetchQueries not updated #222

Merged
merged 1 commit into from May 11, 2022

Conversation

airslice
Copy link
Contributor

@airslice airslice commented May 10, 2022

Overview

The query names used in refetchQuery are not updated, which will lead to some components can't get auto updated after some mutaions.

What I've done

Updated the query names in refetchQuery of hooks:

  • GetAssets
  • GetTeams
  • GetSceneTags
  • GetLayerTags
  • GetProjects

How I tested

Create & remove projects / teams(workspace) / scene tags / layer tags, upload / remove assets of image.

@netlify
Copy link

netlify bot commented May 10, 2022

Deploy Preview for reearth-web ready!

Name Link
🔨 Latest commit ab312e3
🔍 Latest deploy log https://app.netlify.com/sites/reearth-web/deploys/627a81b95ee4080009f7b770
😎 Deploy Preview https://deploy-preview-222--reearth-web.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@codecov
Copy link

codecov bot commented May 10, 2022

Codecov Report

Merging #222 (ab312e3) into main (e290248) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #222   +/-   ##
=======================================
  Coverage   43.99%   43.99%           
=======================================
  Files          63       63           
  Lines        1357     1357           
  Branches      228      228           
=======================================
  Hits          597      597           
  Misses        697      697           
  Partials       63       63           

@airslice airslice marked this pull request as ready for review May 10, 2022 15:44
@airslice airslice requested a review from KaWaite as a code owner May 10, 2022 15:44
Copy link
Member

@KaWaite KaWaite left a comment

Choose a reason for hiding this comment

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

Thanks for doing this so quickly!

@airslice airslice merged commit 7117128 into main May 11, 2022
@airslice airslice deleted the fix/refetch-query-name-not-updated branch May 11, 2022 00:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants