search

Dun-sin / Whisper

A fun Application to have a random chat with people safely
https://whisper.favour.dev/
MIT License
378 stars 342 forks source link

[OTHER] remove the fragment in messageInput #668

Closed Dun-sin closed 1 month ago

Dun-sin commented 1 month ago

What would you like to share?

https://github.com/Dun-sin/Whisper/blob/dbec99c68f63c606647ea2cef1d458a72ae83b1b/client/src/components/Chat/MessageInput.jsx#L61-L118

github-actions[bot] commented 1 month ago

The issue has been unlocked and is now ready for dev. If you would like to work on this issue, you can comment to have it assigned to you.

Tarunkumarkanakam commented 1 month ago

I would like to contribute to this project, Can you brief me on what exactly needs to be done?

Dun-sin commented 1 month ago

I would like to contribute to this project, Can you brief me on what exactly needs to be done?

sure, a react fragment is used to group elements together without using an actually element and it's represented with <> but since that component only has one fragment child we can remove the fragment and just return the form

it's assigned to you, thanks for wanting to contribute. Make sure to read the issue description carefully and ask if you have questions. Follow the rules here, or your PR won't be accepted and will be closed. Good luck!

Dun-sin commented 1 month ago

@Tarunkumarkanakam the time assigned to this Issue is over, you're now unassigned. Thanks for trying to contribute, you're free to ask to work on other issues, if you can meet up with the time assigned.

muhammednithal commented 1 month ago

Can I work on this issue

Dun-sin commented 1 month ago

Can I work on this issue

Sure, thanks for wanting to contribute. Make sure to read the issue description carefully and ask if you have questions. Follow the rules here and keep to the time allocated, or your PR won't be accepted and will be closed. Good luck!