-
-
Notifications
You must be signed in to change notification settings - Fork 798
Add OnDataLoaded EventCallback #8675
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
Conversation
I see @glen-84 you want to review this pull requests, how I can help you to this PR be merge sooner? |
We have put the PR on our internal backlog for discussion @glen-84 will review the actual issue and then we will see what design will fit to your issue. |
I think this have be on another pull request. |
LGTM, I'm just waiting for @michaelstaib to comment on the testing aspect. |
I will create a preview this week and ping you once its available. |
Summary of the changes (Less than 80 chars)
I couldn’t test this because every time I tried to add it to my Blazor project, I got other errors that weren’t related at all.
But based on my experience, I think this is correct.
Please test it if possible.
Also, I couldn’t find UseQuery component tests and didn’t write them. If needed, please guide me.
Additionally, I have a complaint about CONTRIBUTING.md because the information is outdated!
Closes #8673