Skip to content

Comments

feat: Add default group assignment for new users#94

Merged
Classic298 merged 18 commits intodefault-group-assignmentfrom
claude/update-group-endpoints-01Sxhg3SRGFbuuxrq9WrMy74
Nov 20, 2025
Merged

feat: Add default group assignment for new users#94
Classic298 merged 18 commits intodefault-group-assignmentfrom
claude/update-group-endpoints-01Sxhg3SRGFbuuxrq9WrMy74

Conversation

@Classic298
Copy link
Owner

No description provided.

google-labs-jules bot and others added 18 commits September 28, 2025 12:43
This feature allows administrators to configure a default group that new users are automatically added to upon registration.

- Adds a 'Default Group' dropdown in the admin settings.
- Updates the user creation logic to assign the selected default group.
- Includes necessary backend API and database model changes.
Updated the default group assignment feature to work with the new
group_member table and updated API endpoints introduced in the recent
alembic migration (37f288994c47).

Changes:
- Updated Groups.add_user_to_group_by_id() call to use Groups.add_users_to_group()
- Removed obsolete add_user_to_group_by_id() method that used deprecated user_ids column
- Feature now properly integrates with the new GroupMember table structure
- Frontend (General.svelte) requires no changes as it only uses group.id and group.name

This ensures new users are correctly added to the default group during
signup using the proper group members relationship table.
@Classic298 Classic298 changed the base branch from feature/default-group-assignment to default-group-assignment November 20, 2025 22:30
@Classic298 Classic298 merged commit 6388555 into default-group-assignment Nov 20, 2025
@Classic298 Classic298 deleted the claude/update-group-endpoints-01Sxhg3SRGFbuuxrq9WrMy74 branch November 20, 2025 22:34
Classic298 added a commit that referenced this pull request Nov 24, 2025
)

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: Claude <[email protected]>
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