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

Add arg type to RefetchQueriesFunction #7850

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fikrikarim
Copy link

@fikrikarim fikrikarim commented Mar 16, 2021

RefetchQueriesFunction has a FetchResult type on the documentation, but the type is any on the code right now.

This PR add the type to RefetchQueriesFunction according to the documentation and the source code.

If the type is wrong or there is something I need to change, just let me know and I'll try to update it.

Checklist:

  • If this PR is a new feature, please reference an issue where a consensus about the design was reached (not necessary for small changes)
  • Make sure all of the significant new logic is covered by tests

@fikrikarim fikrikarim force-pushed the refetch-queries-function-type branch from 7cdbc3e to 54b3eba Compare March 16, 2021 13:37
@jpvajda jpvajda added 🏓 awaiting-contributor-response requires input from a contributor 🏓 awaiting-team-response requires input from the apollo team and removed 🏓 awaiting-contributor-response requires input from a contributor labels May 3, 2022
@bignimbus
Copy link
Contributor

Hi @fikrikarim 👋🏻 thanks so much for submitting this enhancement! I believe this would be appropriate to consider merging into our 4.0 release given that it's potentially breaking for some users (in a healthy way!) Have a great week and thanks for your patience!

@bignimbus bignimbus added this to the Release 4.0 milestone Sep 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏓 awaiting-team-response requires input from the apollo team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants