Skip to content

Conversation

@Neeraj-gagat
Copy link
Contributor

Description

This PR adds fixes for #317
replaced task.id with task.uuid in edittaskonbackend and modifytaskonbackend functions and updated tests
also replaced task.id with task.uuid in edittask handler or modifytask handler in backend now it these handlers expect task.uuid not task.id

- Fixes: #317

video

Screencast.from.29-12-25.12.38.26.AM.IST.webm

@github-actions
Copy link

Thank you for opening this PR!

Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools.

Please take a moment to:

  • Check the "Files changed" tab
  • Leave comments on any lines for functions, comments, etc. that are important, non-obvious, or may need attention
  • Clarify decisions you made or areas you might be unsure about and/or any future updates being considered.
  • Finally, submit all the comments!

More information on how to conduct a self review:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request

This helps make the review process smoother and gives us a clearer understanding of your thought process.

Once you've added your self-review, we'll continue from our side. Thank you!

Copy link
Contributor Author

@Neeraj-gagat Neeraj-gagat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done with the selfreview every test passes and everything work now user can edit deleted and completed tasks also fixed it by using task.uuid instead of task.id in edittaskonbackend or modifytaskonbackend in frontend and did same changes in backend also in edittask and modifytask handler
and also updated in tests

@its-me-abhishek its-me-abhishek changed the title fixed bug:unable to edit completed and deleted task fix: unable to edit completed and deleted task Dec 28, 2025
@its-me-abhishek its-me-abhishek merged commit 6c2a106 into CCExtractor:main Dec 28, 2025
5 checks passed
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.

Bug: Unable to edit completed or deleted tasks

2 participants