Skip to content
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

fix: use network-only fetchPolicy for tag table #254

Merged

Conversation

loan-laux
Copy link
Contributor

Signed-off-by: Loan Laux [email protected]

Resolves #252
Impact: major
Type: bugfix

Issue

Newly created tags don't show up in the table on the Tags page because of the cache policy. Users need to refresh the page to see their new tags appear, which can be misleading and frustrating.

Solution

Use fetchPolicy="network-only" for the getTags` query.

Breaking changes

None.

Testing

  1. Create a tag
  2. Click "Tags" in the sidebar
  3. Notice that your tag now shows up without the need to refresh the page manually

@HarisSpahija
Copy link

Awesome @loan-laux Thanks!

@willopez willopez self-requested a review April 8, 2020 16:40
Copy link
Member

@willopez willopez left a comment

Choose a reason for hiding this comment

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

Looks good, thanks @loan-laux !

@willopez willopez merged commit 2a956b1 into reactioncommerce:trunk Apr 8, 2020
@kieckhafer kieckhafer mentioned this pull request May 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Creation of Tags only happens after refreshing the page
3 participants