Skip to content

Conversation

@Michi03
Copy link
Contributor

@Michi03 Michi03 commented Feb 25, 2025

This PR adds a button to the RoomList module when the a selectionMode is set. If all of the rooms currently listed are selected the button unselects them. Else, all listed rooms are selected.

Copy link
Member

@tomasr8 tomasr8 left a comment

Choose a reason for hiding this comment

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

Thanks!

@Michi03 Michi03 force-pushed the rb-export-select-all-btn branch from 475cbfe to bb52b07 Compare February 26, 2025 09:08
@GovernmentPlates

This comment was marked as resolved.

@Michi03 Michi03 force-pushed the rb-export-select-all-btn branch from 3ae1757 to 9a6e959 Compare February 28, 2025 12:14
@github-actions github-actions bot added dependencies Pull requests that update a dependency file python Pull requests that update Python deps labels Feb 28, 2025
@Michi03 Michi03 force-pushed the rb-export-select-all-btn branch from 9a6e959 to 52a4fb2 Compare February 28, 2025 12:37
@github-actions github-actions bot removed dependencies Pull requests that update a dependency file python Pull requests that update Python deps labels Feb 28, 2025
@ThiefMaster
Copy link
Member

Works fine and looks nice :)

Just one thought: should we really show this option when you select rooms for blocking? If someone uses this and requests a blocking for 100 rooms, it will be very annoying since they spam every room owner with it...

To me it seems useful for export, but not for blocking.

@ThiefMaster

This comment was marked as resolved.

@Michi03 Michi03 force-pushed the rb-export-select-all-btn branch from 02b446b to 191834d Compare March 4, 2025 08:18
@Michi03 Michi03 force-pushed the rb-export-select-all-btn branch from 02c0bdd to df6d8d4 Compare March 4, 2025 12:39
@Michi03 Michi03 requested a review from GovernmentPlates March 4, 2025 12:53
@Michi03 Michi03 force-pushed the rb-export-select-all-btn branch from df6d8d4 to d6298dd Compare March 10, 2025 08:23
@ThiefMaster ThiefMaster force-pushed the rb-export-select-all-btn branch from d6298dd to c65ef6d Compare March 18, 2025 16:29
Copy link
Member

@ThiefMaster ThiefMaster left a comment

Choose a reason for hiding this comment

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

thx!

@ThiefMaster ThiefMaster force-pushed the rb-export-select-all-btn branch from c65ef6d to 90ad517 Compare March 18, 2025 16:34
@ThiefMaster ThiefMaster force-pushed the rb-export-select-all-btn branch from 90ad517 to b58bd0c Compare March 18, 2025 16:35
@ThiefMaster ThiefMaster added this to the v3.3 milestone Mar 18, 2025
@ThiefMaster ThiefMaster enabled auto-merge (squash) March 18, 2025 16:35
@ThiefMaster ThiefMaster merged commit 7d0945a into indico:master Mar 18, 2025
10 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.

4 participants