-
Notifications
You must be signed in to change notification settings - Fork 10.9k
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
[NEW] useUserData Hook #20584
[NEW] useUserData Hook #20584
Conversation
This pull request introduces 1 alert when merging a902fd5 into 61ca8bf - view on LGTM.com new alerts:
|
Just a question, can we use this hook to get the custom user status text? I had made Pull Request #20257 (Open) a while back where I had to make a few changes to get the live user status text. |
hey @aditya-mitra (actually we have more issues related with that we will be solved having that hook) |
@ggazzo Or should I wait until this pull request is merged? Thank you. |
hm... wait a minute I will merge this one today/tomorrow no more, I like the idea to keep it in different prs, AND I think I would make change few things in your pr if I could |
Sure. |
15eb4ef
to
d26ebcc
Compare
Proposed changes (including videos or screenshots)
Issue(s)
Steps to test or reproduce
Further comments