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

I need a queryOptions prop to be added to useReferenceInputController and ReferenceInput for request that fetches current value. #9192

Closed
ethembynkr opened this issue Aug 17, 2023 · 3 comments · Fixed by #9197
Assignees
Labels

Comments

@ethembynkr
Copy link
Contributor

ethembynkr commented Aug 17, 2023

Is your feature request related to a problem? Please describe.
In useReferenceInputController, I can pass options to query that fetches possible values, but not to the query that fetches current value.

Describe the solution you'd like
I need a queryOptions prop to be added to useReferenceInputController and ReferenceInput for request that fetches current value.

Additional context
image

@ethembynkr ethembynkr changed the title I need the queryOptions prop for request that fetches the current value in useReferenceInputController. I need a queryOptions prop to be added to useReferenceInputController for request that fetches current value. Aug 17, 2023
@ethembynkr ethembynkr changed the title I need a queryOptions prop to be added to useReferenceInputController for request that fetches current value. I need a queryOptions prop to be added to useReferenceInputController and ReferenceInput for request that fetches current value. Aug 18, 2023
@fzaninotto
Copy link
Member

We know that the queryOptions is only passed to one of the two queries. But we've wondered if passing the same options to the two queries would be a good idea. For once, the types aren't the same, so TypeScript complains. But mostly, I don't see any use case for this - we're already passing meta to the two queries.

Could you explain why you need this?

@ethembynkr
Copy link
Contributor Author

We want to pass staleTime to both queries.

@fzaninotto
Copy link
Member

Good enough reason! I'll mark this as a bug then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants