Skip to content

Conversation

@aheizi
Copy link
Contributor

@aheizi aheizi commented Mar 12, 2025

Context

Add the function of batch deleting historical tasks

Implementation

Add selection mode and batch delete historical tasks

Screenshots

image

How to Test

manual test

Get in Touch

aheizi


Important

Adds batch delete functionality for historical tasks with UI support for selection mode and batch delete confirmation.

  • Behavior:
    • Adds batch delete functionality for historical tasks in ClineProvider.
    • Implements batch processing in deleteMultipleTasksWithIds to handle tasks in batches of 20.
    • Updates UI after each batch to show progress.
  • UI Components:
    • Adds BatchDeleteTaskDialog component for batch delete confirmation.
    • Updates HistoryView to support selection mode and batch delete actions.
    • Adds selection controls and batch delete button in HistoryView.
  • Localization:
    • Updates localization files (history.json) for new batch delete strings in multiple languages.

This description was created by Ellipsis for f60feec. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Mar 12, 2025

⚠️ No Changeset found

Latest commit: 7433392

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Mar 12, 2025
@cte
Copy link
Collaborator

cte commented Mar 12, 2025

Nice - thank you!

@dosubot dosubot bot added the enhancement New feature or request label Mar 12, 2025
@aheizi aheizi force-pushed the batch_clear_history branch from 6b787da to d088dda Compare March 12, 2025 15:52
@mrubens
Copy link
Collaborator

mrubens commented Mar 12, 2025

I love this! I just tried deleting my 421 tasks and it didn't end up deleting all of them though. Is there some limit?

Oh, maybe this error?
Screenshot 2025-03-12 at 12 05 10 PM

@aheizi
Copy link
Contributor Author

aheizi commented Mar 13, 2025

I love this! I just tried deleting my 421 tasks and it didn't end up deleting all of them though. Is there some limit?

Oh, maybe this error? Screenshot 2025-03-12 at 12 05 10 PM

@mrubens It looks like a problem where the task cannot be found, let me solve it.

i see the code
image

I will try to tolerate it too

@mrubens
Copy link
Collaborator

mrubens commented Mar 13, 2025

Hmm, for some reason it's not deleting any of my tasks now

@aheizi
Copy link
Contributor Author

aheizi commented Mar 13, 2025

Hmm, for some reason it's not deleting any of my tasks now

@mrubens I can't reproduce this problem here, can you help provide more clues ?

@mrubens
Copy link
Collaborator

mrubens commented Mar 13, 2025

Hmm, for some reason it's not deleting any of my tasks now

@mrubens I can't reproduce this problem here, can you help provide more clues ?

Yeah I will step through it in a bit

@hannesrudolph hannesrudolph moved this from New to PR [Greenlit] in Roo Code Roadmap Mar 13, 2025
@aheizi
Copy link
Contributor Author

aheizi commented Mar 16, 2025

Hmm, for some reason it's not deleting any of my tasks now

@mrubens I can't reproduce this problem here, can you help provide more clues ?

Yeah I will step through it in a bit

@mrubens I tried to fix this issue, could you please check if the problem can be reproduced in your environment?

aheizi added 6 commits March 16, 2025 13:36
# Conflicts:
#	webview-ui/src/components/history/HistoryView.tsx
# Conflicts:
#	webview-ui/src/components/history/HistoryView.tsx
#	webview-ui/src/i18n/__mocks__/TranslationContext.tsx
#	webview-ui/src/i18n/locales/ar/common.json
#	webview-ui/src/i18n/locales/ar/history.json
#	webview-ui/src/i18n/locales/ca/common.json
#	webview-ui/src/i18n/locales/cs/common.json
#	webview-ui/src/i18n/locales/cs/history.json
#	webview-ui/src/i18n/locales/de/common.json
#	webview-ui/src/i18n/locales/en/common.json
#	webview-ui/src/i18n/locales/es/common.json
#	webview-ui/src/i18n/locales/fr/common.json
#	webview-ui/src/i18n/locales/hi/common.json
#	webview-ui/src/i18n/locales/hu/common.json
#	webview-ui/src/i18n/locales/hu/history.json
#	webview-ui/src/i18n/locales/it/common.json
#	webview-ui/src/i18n/locales/ja/common.json
#	webview-ui/src/i18n/locales/ko/common.json
#	webview-ui/src/i18n/locales/pl/common.json
#	webview-ui/src/i18n/locales/pt-BR/common.json
#	webview-ui/src/i18n/locales/pt/common.json
#	webview-ui/src/i18n/locales/pt/history.json
#	webview-ui/src/i18n/locales/ru/common.json
#	webview-ui/src/i18n/locales/ru/history.json
#	webview-ui/src/i18n/locales/tr/common.json
#	webview-ui/src/i18n/locales/zh-CN/common.json
#	webview-ui/src/i18n/locales/zh-TW/common.json
Comment on lines 2 to 4
"common": {
"cancel": "Cancel·lar"
},
Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like we already have "cancel" translated blow on line 23/28

Copy link
Collaborator

@mrubens mrubens left a comment

Choose a reason for hiding this comment

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

One nitpick on the translations, but not a blocker

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 18, 2025
aheizi added 4 commits March 18, 2025 14:00
# Conflicts:
#	webview-ui/src/components/history/HistoryView.tsx
# Conflicts:
#	src/core/webview/ClineProvider.ts
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Mar 18, 2025
@aheizi aheizi force-pushed the batch_clear_history branch from f60feec to 29351a1 Compare March 18, 2025 08:09
@aheizi aheizi marked this pull request as draft March 18, 2025 08:12
@aheizi aheizi force-pushed the batch_clear_history branch from 29351a1 to f60feec Compare March 18, 2025 08:19
@aheizi aheizi marked this pull request as ready for review March 18, 2025 08:20
@ellipsis-dev
Copy link
Contributor

ellipsis-dev bot commented Mar 18, 2025

This pull request includes a significant number of changes across multiple files, including both documentation updates and new feature implementations. To improve the review process, I suggest splitting this pull request into two separate ones:

  1. Documentation Updates: This would include all the changes related to the reformatting of contributors' tables in the README files across different languages.

  2. Batch Deletion Feature: This would focus on the new feature implementation for batch deletion of tasks, including changes in the core logic and UI components.

By separating these changes, it will be easier to review and test each set of changes independently. Thank you!

@mrubens mrubens merged commit 7ef4841 into RooCodeInc:main Mar 18, 2025
10 checks passed
@github-project-automation github-project-automation bot moved this from PR [Greenlit] to Done in Roo Code Roadmap Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request lgtm This PR has been approved by a maintainer size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants