Skip to content

Enable multiple user groups for team organizations#14875

Open
PabloBuchu wants to merge 2 commits intodevelopfrom
wip/pb/enable-multiple-user-groups
Open

Enable multiple user groups for team organizations#14875
PabloBuchu wants to merge 2 commits intodevelopfrom
wip/pb/enable-multiple-user-groups

Conversation

@PabloBuchu
Copy link
Contributor

Pull Request Description

This PR:

  • enables team organizations to have up to 10 user groups
  • fixes drag n dropping between teams categories
  • improves texts when creating new user group

Important Notes

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

  • The documentation has been updated, if necessary.
  • Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
  • All code follows the
    Scala,
    Java,
    TypeScript,
    and
    Rust
    style guides. In case you are using a language not listed above, follow the Rust style guide.
  • Unit tests have been written where possible.
  • If meaningful changes were made to logic or tests affecting Enso Cloud integration in the libraries,
    or the Snowflake database integration, a run of the Extra Tests has been scheduled.
    • If applicable, it is suggested to paste a link to a successful run of the Extra Tests.

@PabloBuchu PabloBuchu added the CI: No changelog needed Do not require a changelog entry for this PR. label Mar 16, 2026
@PabloBuchu PabloBuchu changed the title Wip/pb/enable multiple user groups Enable multiple user groups for team organizations Mar 16, 2026
parentId: DirectoryId | null = null,
) {
const operation = dropOperationBetweenCategories(from, to, parentId)
console.log(operation)
Copy link
Contributor

Choose a reason for hiding this comment

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

debug log leftover

Copy link
Contributor

Choose a reason for hiding this comment

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

I recommend usingconsole.debug for such "debug logs" - these are caught by linter, so won't be merged.

"newUserGroup": "New User Group",
"userGroupsPaywallMessage": "You have reached the limit of user groups for your plan. Upgrade to create more user groups.",
"userGroupsLimitMessage": "You can create up to $0 user group(s)",
"userGroupsLimitMessage": "You can create up to $0 user group(s). $1 left.",
Copy link
Contributor

Choose a reason for hiding this comment

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

linter fails because type signature needs to be updated for this message (search for userGroupsLimitMessage)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: No changelog needed Do not require a changelog entry for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants