-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 Member #4565
Add Member #4565
Conversation
🚀 Preview for commit de109cc at: https://64b623d8374e6d28373d9926--layer5.netlify.app |
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.
LGTM
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, nice work @devarsh10
But can you please change the title of the PR? It is not relevant to what issue you are working on.
@HarshDobariya79 can you review this so we can call for the final checks to get this PR merged? |
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.
Thank you @devarsh10 for the PR. Nice work.
My pleasure Harsh. |
🚀 Preview for commit 177fa98 at: https://64b7e42106e65a1272e006c7--layer5.netlify.app |
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.
Yay! 🎈 Harsh 🎸
Thank you @devarsh10 😎
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.
Let's go. 🚀
Description
Signed-off-by: devarsh10 [email protected]
This PR fixes #4556
Notes for Reviewers
Add @HarshDobariya47 as a Member.