fix

Author: matusdrobuliak66

services/web/server/src/simcore_service_webserver/conversations/_controller/_conversations_messages_rest.py

@@ -118,9 +118,9 @@ async def create_conversation_message(request: web.Request):
             )
             _url = request.url
             if _url.port:
-                _conversation_url = f"{_url.scheme}://{_url.host}:{_url.port}/#/conversations/{path_params.conversation_id}"
+                _conversation_url = f"{_url.scheme}://{_url.host}:{_url.port}/#/conversation/{path_params.conversation_id}"
             else:
-                _conversation_url = f"{_url.scheme}://{_url.host}/#/conversations/{path_params.conversation_id}"
+                _conversation_url = f"{_url.scheme}://{_url.host}/#/conversation/{path_params.conversation_id}"
             _extra_context = _conversation.extra_context
             await email_service.send_email_from_template(
                 request,

services/web/server/src/simcore_service_webserver/groups/_groups_repository.py

@@ -550,12 +550,16 @@ async def list_users_in_group(
 
         # Get all users in the group
         query = (
-            sa.select(*_GROUP_COLUMNS)
-            .select_from(
-                groups.join(
-                    user_to_groups, user_to_groups.c.gid == groups.c.gid, isouter=True
-                ).join(users, users.c.id == user_to_groups.c.uid)
+            sa.select(
+                users.c.id,
+                users.c.name,
+                users.c.email,
+                users.c.first_name,
+                users.c.last_name,
+                users.c.primary_gid,
+                # user_to_groups.c.access_rights,  # <-- currently not neccessary, might be added if needed
             )
+            .select_from(users.join(user_to_groups, users.c.id == user_to_groups.c.uid))
             .where(user_to_groups.c.gid == group_id)
         )
 

services/web/server/tests/unit/with_dbs/01/groups/test_groups_repository.py

@@ -0,0 +1,51 @@
+# pylint: disable=redefined-outer-name
+# pylint: disable=too-many-arguments
+# pylint: disable=unused-argument
+# pylint: disable=unused-variable
+
+import pytest
+from aiohttp.test_utils import TestClient
+from models_library.groups import GroupMember, StandardGroupCreate
+from pytest_simcore.helpers.webserver_users import UserInfoDict
+from simcore_postgres_database.models.users import UserRole
+from simcore_service_webserver.groups._groups_repository import (
+    create_standard_group,
+    delete_standard_group,
+    list_users_in_group,
+)
+
+
+@pytest.mark.parametrize("user_role", [UserRole.USER])
+async def test_list_users_in_group_owner_only(
+    client: TestClient,
+    user_role: UserRole,
+    logged_user: UserInfoDict,
+):
+    """Test list_users_in_group returns only the owner for a new group."""
+    assert client.app
+
+    # Create a standard group
+    group, _ = await create_standard_group(
+        app=client.app,
+        user_id=logged_user["id"],
+        create=StandardGroupCreate(
+            name="Test Owner Only Group",
+            description="A test group to check owner-only user list",
+            thumbnail=None,
+        ),
+    )
+
+    try:
+        # List users in the group - should only contain the owner
+        users_in_group = await list_users_in_group(app=client.app, group_id=group.gid)
+
+        # Should contain exactly one user (the owner)
+        assert len(users_in_group) == 1
+        assert isinstance(users_in_group[0], GroupMember)
+        assert users_in_group[0].id == logged_user["id"]
+
+    finally:
+        # Clean up
+        await delete_standard_group(
+            app=client.app, user_id=logged_user["id"], group_id=group.gid
+        )