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

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

Closed
2 of 4 tasks
narayan954 opened this issue Sep 30, 2023 · 7 comments · Fixed by #1276
Closed
2 of 4 tasks

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

narayan954 opened this issue Sep 30, 2023 · 7 comments · Fixed by #1276

Comments

@narayan954
Copy link
Owner

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

  • I agree to follow this project's Code of Conduct
  • I have checked the existing issues
  • I'm a GSSoC'23 contributor
  • I want to work on this issue
@github-actions
Copy link

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
Copy link

4genie commented Oct 6, 2023

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

@narayan954
Copy link
Owner Author

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

Sure, you can go ahead! :)

@Jai-Marothiya
Copy link
Contributor

Hi @narayan954,

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

Thanks

@narayan954
Copy link
Owner Author

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! :)

@narayan954 narayan954 assigned Jai-Marothiya and unassigned 4genie Oct 13, 2023
@Jai-Marothiya
Copy link
Contributor

Hi @narayan954,

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

Thanks.

@narayan954
Copy link
Owner Author

Hi @narayan954,

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

Thanks.

reviewing :)

@narayan954 narayan954 linked a pull request Oct 16, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants