-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Fix: Memoize user profile for performance improvement #5158
Fix: Memoize user profile for performance improvement #5158
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's great having you contribute to this project
Welcome to the community 🤓If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.
Welcome @psavan1655! Thank you so much for your first pull request! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
Hi @psavan1655 |
Reviewpad Report 📜 Executed actions $addLabel("small")
$removeLabels(["medium", "large"])
$commentOnce($sprintf("Welcome @%v! Thank you so much for your first pull request!", [$author()])) |
Hi @FrozenHearth, Thanks for pointing out, WIll surely take a look into it. @eddiejaoude Should i work on it? I have reproduced the issue and verified it. |
Sure, but please also raise an issue (bug) |
#5535 Created this issue. You can assign me there. |
Fixes Issue
Closes #5003
Changes proposed
Check List (Check all the applicable boxes)
Screenshots
https://www.loom.com/share/c0b4d8c001734d86931668b43d53e51f
Note to reviewers