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

Add group_members feature flag #9062

Merged
merged 2 commits into from
Nov 7, 2024
Merged

Add group_members feature flag #9062

merged 2 commits into from
Nov 7, 2024

Conversation

robertknight
Copy link
Member

@robertknight robertknight commented Nov 1, 2024

This is a flag to control whether the new group forms display the members table.

Part of #9063.

This doesn't affect order in the admin dashboard, as flags were already
sorted by name there. It just makes it easier to skim the list.
This a flag to control whether the new group forms display the members table.
"pdf_custom_text_layer": "Use custom text layer in PDFs for improved text selection",
"styled_highlight_clusters": "Style different clusters of highlights in the client",
"client_user_profile": "Enable client-side user profile and preferences management",
"group_type": "Allow users to choose group type in group creation form",
Copy link
Member Author

Choose a reason for hiding this comment

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

While adding this flag I sorted the existing ones by name to make it more obvious where to slot new entries in. Looking at the rest of the code in this file it shouldn't matter what order the entries are in, and the admin UI already sorted flags for display, so there is no change there.

Copy link
Contributor

Choose a reason for hiding this comment

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

👍

"pdf_custom_text_layer": "Use custom text layer in PDFs for improved text selection",
"styled_highlight_clusters": "Style different clusters of highlights in the client",
"client_user_profile": "Enable client-side user profile and preferences management",
"group_type": "Allow users to choose group type in group creation form",
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@robertknight robertknight merged commit 76c74f7 into main Nov 7, 2024
9 checks passed
@robertknight robertknight deleted the group-members-flag branch November 7, 2024 12:12
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