Skip to content

fix: runner select all#8256

Merged
CurryYangxx merged 1 commit into
developfrom
fix/runner-select-all
Feb 17, 2025
Merged

fix: runner select all#8256
CurryYangxx merged 1 commit into
developfrom
fix/runner-select-all

Conversation

@CurryYangxx
Copy link
Copy Markdown
Member

a keyboard shortcut ⌘ Cmd + A can select all grid list items and let selectedKeys to be all, we need to handle this case

@CurryYangxx CurryYangxx requested a review from ihexxa December 18, 2024 08:29
@@ -476,10 +476,10 @@ export const Runner: FC<{}> = () => {
? Array.from(reqList.items)
.filter(item => item.ancestorIds.includes(targetFolderId))
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

is this filter necessary?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It looks like we can drop it.

ihexxa
ihexxa previously approved these changes Dec 18, 2024
Copy link
Copy Markdown
Contributor

@ihexxa ihexxa left a comment

Choose a reason for hiding this comment

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

Thanks for the fix 👍 .

@@ -476,10 +476,10 @@ export const Runner: FC<{}> = () => {
? Array.from(reqList.items)
.filter(item => item.ancestorIds.includes(targetFolderId))
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It looks like we can drop it.

@CurryYangxx CurryYangxx marked this pull request as draft December 18, 2024 09:18
@CurryYangxx
Copy link
Copy Markdown
Member Author

will merge after 10.3

@CurryYangxx CurryYangxx force-pushed the fix/runner-select-all branch from 245193e to e271c50 Compare December 26, 2024 09:02
@CurryYangxx CurryYangxx force-pushed the fix/runner-select-all branch 2 times, most recently from 519bda0 to d50513f Compare January 8, 2025 08:24
@CurryYangxx CurryYangxx force-pushed the fix/runner-select-all branch from d50513f to a11c20b Compare February 17, 2025 08:32
@CurryYangxx CurryYangxx force-pushed the fix/runner-select-all branch from a11c20b to 43eeb00 Compare February 17, 2025 09:19
@CurryYangxx CurryYangxx marked this pull request as ready for review February 17, 2025 09:19
@CurryYangxx CurryYangxx enabled auto-merge (squash) February 17, 2025 09:28
@CurryYangxx CurryYangxx merged commit 5e47147 into develop Feb 17, 2025
@CurryYangxx CurryYangxx deleted the fix/runner-select-all branch February 17, 2025 09:38
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.

2 participants