bug: deleting comment deletes oldest comment instead of selected one

narayan954 / dummygram

DummyGram is a social media application where we have posts, stories, chat, friends, authentication, comment, delete, share and like feature. It uses ReactJs in the frontend for the UI and Firebase for the backend storing the users, posts, stories and comments.

https://narayan954.github.io/dummygram/

MIT License

76 stars 185 forks source link

bug: deleting comment deletes oldest comment instead of selected one #1267

Closed narayan954 closed 10 months ago

narayan954 commented 11 months ago

Description

deleting comment deletes oldest comment instead of selected one. Some issue in delete function.

Steps to reproduce

craft two or more comments in a post. delete any of them, it'll only delete the oldest one

Screenshots

No response

Desktop (please complete the following information)

No response

Smartphone (please complete the following information)

No response

Additional information

No response

Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[ ] I'm a GSSoC'23 contributor
[ ] I want to work on this issue

github-actions[bot] commented 11 months ago

Hello narayan954! Thank you for raising this issue! 😊 Your contribution is valuable to us! 😊

Please make sure to follow our Contributing Guidelines. 💪🏻

Our reviewers shall carefully assess the issue and reach out to you soon! 😇 We appreciate your patience!

4genie commented 11 months ago

I want to work on this issue,please assign to me

narayan954 commented 11 months ago

I want to work on this issue,please assign to me

Sure, you can go ahead! :)

Jai-Marothiya commented 11 months ago

Hi @narayan954,

I go through your code and I think I can fix the issue. kindly assign this task to me.

Thanks

narayan954 commented 10 months ago

Hi @narayan954,

I go through your code and I think I can fix the issue. kindly assign this task to me.

Thanks

Hi @Jai-Marothiya you can go ahead! :)

Jai-Marothiya commented 10 months ago

Hi @narayan954,

I created a pull request for this issue, please merge it and mark it as Hacktoberfest accepted.

Thanks.

narayan954 commented 10 months ago

Hi @narayan954,

I created a pull request for this issue, please merge it and mark it as Hacktoberfest accepted.

Thanks.

reviewing :)