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

Feature/refactor js to ts #71

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kits4
Copy link

@kits4 kits4 commented Oct 16, 2021

I have solved issue #53
please do review my code and inform me if any changes are required.
I have removed some default props from Chat msg compoent.

Thank You!!!

@Rishabh-malhotraa
Copy link
Owner

can you allow edits from maintainer

https://docs.github.com/en/github/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork

Thank you so much for the pull request, I am really really busy with my college I'll try to review this and get this merged by the end of the week

@kits4
Copy link
Author

kits4 commented Oct 17, 2021

It's already done can you please verify it.
image

@kits4
Copy link
Author

kits4 commented Oct 24, 2021

can you please verify this pr after which it can be added to hacktoberfest.

@Rishabh-malhotraa Rishabh-malhotraa force-pushed the main branch 15 times, most recently from 5dcd1f7 to 2a7e2a6 Compare April 29, 2024 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants