Merge remote-tracking branch 'origin/develop' into backend/feature/reorder-user-communities

Author: kevinortiz43

.vscode/settings.json

@@ -55,7 +55,6 @@
   "editor.tabSize": 2,
   "editor.insertSpaces": true,
   "files.trimTrailingWhitespace": true,
-  "eslint.useFlatConfig": false,
   "python-env.workspaceSearchPaths": [ "app/backend/.venv" ],
   "python.testing.cwd": "app/backend/src/tests",
   "python.testing.unittestEnabled": false,

app/backend/src/couchers/email/locales/it.json

@@ -2,6 +2,12 @@
   "generic": {
     "greeting_line": "Ciao {{name}},",
     "security_warning_contact_support": "Se non sei stato tu, contattaci a {{ email_link }} e risolveremo il prima possibile.",
-    "do_not_reply_request": "Non rispondere a questa mail. Usa uno dei pulsanti sopra per rispondere a questa richiesta."
+    "do_not_reply_request": "Non rispondere a questa mail. Usa uno dei pulsanti sopra per rispondere a questa richiesta.",
+    "thanks": "Grazie!",
+    "closing_lines": {
+      "default": "Saluti, <br>Il Team Couchers.org",
+      "founders": "Aapeli and Itsi,<br>Fondatori Couchers.org",
+      "aapeli": "Aapeli<br>Co-fondatore Couchers.org<br>{{ profile_link }}"
+    }
   }
 }

app/backend/src/couchers/i18n/locales/en.json

@@ -98,6 +98,7 @@
     "incomplete_profile_create_discussion": "You have to complete your profile before you can start a discussion.",
     "incomplete_profile_create_event": "You have to complete your profile before you can create an event.",
     "incomplete_profile_create_public_trip": "You have to complete your profile before you can create a public trip.",
+    "incomplete_profile_post_comment": "You have to complete your profile before you can post a comment.",
     "incomplete_profile_send_friend_request": "You have to complete your profile before you can send a friend request.",
     "incomplete_profile_send_message": "You have to complete your profile before you can send a message.",
     "incomplete_profile_send_request": "You have to complete your profile before you can send a request.",

app/backend/src/couchers/i18n/locales/it.json

@@ -19,6 +19,8 @@
     "date_to_after_one_year": "Non puoi chiedere di essere ospitato per più di un anno.",
     "discussion_not_found": "Discussione non trovata.",
     "account_not_found": "Non è stato trovato alcun account con questo username o email.",
-    "address_line_1_required": "La linea 1 dell'indirizzo è obbligatoria."
+    "address_line_1_required": "La linea 1 dell'indirizzo è obbligatoria.",
+    "city_required": "La città è obbligatoria.",
+    "already_in_chat": "L'utente è già nella chat."
   }
 }

app/backend/src/couchers/i18n/locales/zh-Hans.json

@@ -1,7 +1,7 @@
 {
   "errors": {
-    "account_not_found": "未找到使用该用户名或邮箱的账户。",
-    "account_suspended": "您的账户已被停用。",
+    "account_not_found": "未找到使用该用户名或邮箱的账号。",
+    "account_suspended": "您的账号已被暂停使用。",
     "address_line_1_required": "必须填写地址第1行。",
     "already_admin": "该用户已经是管理员。",
     "already_have_dm": "您已经和该用户有私信聊天。",
@@ -18,7 +18,7 @@
     "cannot_leave_containing_community": "您的个人资料位置位于该社区内，因此您不能退出。不过，您可以在账户设置中调整通知。",
     "cant_add_self": "您不能把自己加入群聊。",
     "cant_block_self": "您不能屏蔽自己。",
-    "cant_friend_self": "您不能和自己成为好友！",
+    "cant_friend_self": "您无法将自己添加为好友！",
     "cant_invite_self": "您不能邀请自己进入聊天。",
     "cant_invite_to_dm": "您不能邀请其他用户加入私信聊天。",
     "can_only_refer_friends": "您只能为已确认的好友写评语。",

app/backend/src/couchers/notifications/locales/it.json

@@ -2,21 +2,37 @@
   "account_deletion": {
     "complete": {
       "push": {
-        "title": "Account cancellato"
-      }
+        "title": "Account cancellato",
+        "ios_title": "Account eliminato",
+        "body_one": "Puoi recuperarlo entro {{count}} giorno usando il link che ti abbiamo inviato.",
+        "body_many": "Puoi recuperarlo entro {{count}} di giorni usando il link che ti abbiamo inviato.",
+        "body_other": "Puoi recuperarlo entro {{count}} giorni usando il link che ti abbiamo inviato."
+      },
+      "event_description": "Il tuo account è stato eliminato"
     },
     "start": {
       "push": {
         "ios_title": "E' stata richiesta la cancellazione dell'account",
-        "body": "Usa il link che ti abbiamo inviato via email per confermare."
+        "body": "Usa il link che ti abbiamo inviato via email per confermare.",
+        "title": "Eliminazione account richiesta"
+      },
+      "event_description": "Hai iniziato l'eliminazione dell'account"
+    },
+    "recovered": {
+      "event_description": "Il tuo account è stato ripristinato (non è stato cancellato)",
+      "push": {
+        "title": "Account ripristinato",
+        "ios_title": "Accout ripristinato",
+        "body": "Bentornato!"
       }
     }
   },
   "_adhoc": {
     "push_enabled": {
       "push": {
         "title": "Notifiche push abilitate",
-        "body": "Da adesso riceverai le notifiche su questo dispositivo."
+        "body": "Da adesso riceverai le notifiche su questo dispositivo.",
+        "ios_title": "Notifiche push abilitate"
       }
     }
   }

app/backend/src/couchers/servicers/threads.py

@@ -9,6 +9,7 @@
 
 from couchers.context import CouchersContext, make_background_user_context, make_notification_user_context
 from couchers.db import session_scope
+from couchers.helpers.completed_profile import has_completed_profile
 from couchers.jobs.enqueue import queue_job
 from couchers.models import (
     Comment,
@@ -385,6 +386,10 @@ def PostReply(
         if depth not in (0, 1):
             context.abort_with_error_code(grpc.StatusCode.NOT_FOUND, "thread_not_found")
 
+        user = session.execute(select(User).where(User.id == context.user_id)).scalar_one()
+        if not has_completed_profile(session, user):
+            context.abort_with_error_code(grpc.StatusCode.FAILED_PRECONDITION, "incomplete_profile_post_comment")
+
         object_to_add: Comment | Reply | None = None
 
         def create_object(moderation_state_id: int) -> int:

app/backend/src/tests/test_threads.py

@@ -1,5 +1,6 @@
 import string
 import textwrap
+from datetime import timedelta
 
 import grpc
 import pytest
@@ -17,11 +18,12 @@
     User,
 )
 from couchers.models.discussions import CommentVersion, ContentChangeType, ReplyVersion
-from couchers.proto import moderation_pb2, threads_pb2
+from couchers.proto import discussions_pb2, events_pb2, moderation_pb2, threads_pb2
 from couchers.servicers.threads import pack_thread_id
-from couchers.utils import now
+from couchers.utils import Timestamp_from_datetime, now
 from tests.fixtures.db import generate_user
-from tests.fixtures.sessions import real_moderation_session, threads_session
+from tests.fixtures.sessions import discussions_session, events_session, real_moderation_session, threads_session
+from tests.test_communities import create_community
 
 
 @pytest.fixture(autouse=True)
@@ -539,6 +541,101 @@ def test_delete_reply_creates_version_record(db):
         assert v.editor_user_id == user.id
 
 
+def _create_event_and_get_thread_id(organizer, organizer_token: str) -> int:
+    """Helper: create an Event via the API, return its (packed) thread_id."""
+    with session_scope() as session:
+        create_community(session, 0, 2, "Testing Community", [organizer], [], None)
+
+    start_time = now() + timedelta(hours=2)
+    end_time = start_time + timedelta(hours=3)
+    with events_session(organizer_token) as api:
+        res = api.CreateEvent(
+            events_pb2.CreateEventReq(
+                title="Dummy event",
+                content="Dummy content.",
+                location=events_pb2.EventLocation(
+                    address="Near Null Island",
+                    lat=0.1,
+                    lng=0.2,
+                ),
+                start_time=Timestamp_from_datetime(start_time),
+                end_time=Timestamp_from_datetime(end_time),
+                timezone="UTC",
+            )
+        )
+    return int(res.thread.thread_id)
+
+
+def test_post_reply_incomplete_profile_blocked_on_event_comment(db):
+    """An incomplete-profile user cannot post a top-level comment on an event's thread."""
+    organizer, organizer_token = generate_user()
+    _commenter, commenter_token = generate_user(complete_profile=False)
+
+    event_thread_id = _create_event_and_get_thread_id(organizer, organizer_token)
+
+    with threads_session(commenter_token) as api:
+        with pytest.raises(grpc.RpcError) as e:
+            api.PostReply(threads_pb2.PostReplyReq(thread_id=event_thread_id, content="hello"))
+        assert e.value.code() == grpc.StatusCode.FAILED_PRECONDITION
+
+
+def test_post_reply_incomplete_profile_blocked_on_event_reply(db):
+    """An incomplete-profile user cannot post a nested reply within an event's thread."""
+    organizer, organizer_token = generate_user()
+    _commenter, commenter_token = generate_user()
+    _replier, replier_token = generate_user(complete_profile=False)
+
+    event_thread_id = _create_event_and_get_thread_id(organizer, organizer_token)
+
+    with threads_session(commenter_token) as api:
+        comment_thread_id = api.PostReply(
+            threads_pb2.PostReplyReq(thread_id=event_thread_id, content="top-level comment")
+        ).thread_id
+
+    with threads_session(replier_token) as api:
+        with pytest.raises(grpc.RpcError) as e:
+            api.PostReply(threads_pb2.PostReplyReq(thread_id=comment_thread_id, content="a reply"))
+        assert e.value.code() == grpc.StatusCode.FAILED_PRECONDITION
+
+
+def test_post_reply_complete_profile_allowed_on_event_comment(db):
+    """A complete-profile user can post a comment on an event's thread."""
+    organizer, organizer_token = generate_user()
+    _commenter, commenter_token = generate_user()
+
+    event_thread_id = _create_event_and_get_thread_id(organizer, organizer_token)
+
+    with threads_session(commenter_token) as api:
+        comment_thread_id = api.PostReply(
+            threads_pb2.PostReplyReq(thread_id=event_thread_id, content="hello")
+        ).thread_id
+    assert comment_thread_id
+
+
+def test_post_reply_incomplete_profile_blocked_on_discussion_thread(db):
+    """An incomplete-profile user cannot post a comment on a discussion thread."""
+    admin, admin_token = generate_user()
+    _commenter, commenter_token = generate_user(complete_profile=False)
+
+    with session_scope() as session:
+        community_id = create_community(session, 0, 1, "Testing Community", [admin], [], None).id
+
+    with discussions_session(admin_token) as api:
+        discussion = api.CreateDiscussion(
+            discussions_pb2.CreateDiscussionReq(
+                title="A discussion",
+                content="Content",
+                owner_community_id=community_id,
+            )
+        )
+        discussion_thread_id = discussion.thread.thread_id
+
+    with threads_session(commenter_token) as api:
+        with pytest.raises(grpc.RpcError) as e:
+            api.PostReply(threads_pb2.PostReplyReq(thread_id=discussion_thread_id, content="hello"))
+        assert e.value.code() == grpc.StatusCode.FAILED_PRECONDITION
+
+
 def test_edit_comment_multiple_edits_creates_multiple_version_records(db):
     user, token = generate_user()
     _, comment_thread_id = _make_thread_and_comment(token, content="v1")

app/i18n/glossary/it.csv

@@ -1,4 +1,4 @@
 "source","target","context","translator_comments","developer_comments"
 "Event","Evento","","",""
-"blog","","","",""
-"Strong verification","","","",""
+"blog","blog","","",""
+"Strong verification","Verifica robusta","","",""

app/mobile/i18n/locales/it.json

@@ -2,6 +2,43 @@
   "errors": {
     "check_connection": "Controlla la tua connessione Internet e riprova.",
     "try_again": "Riprova",
-    "failed_to_load": "Errore di caricamento"
+    "failed_to_load": "Errore di caricamento",
+    "camera_permission_denied": "Per fare le foto è necessario fornire l'autorizzazione della fotocamera.",
+    "camera_permission_required": "Permesso fotocamera richiesto",
+    "camera_permission_explanation": "Questa applicazione ha bisogno dell'accesso alla fotocamera per fare le foto. Per favore concedere il permesso quando richiesto.",
+    "camera_permission_denied_permanently": "L'accesso alla fotocamera è stato negato. Per fare le foto dovrai abilitare il permesso della fotocamera nelle impostazioni del tuo dispositivo."
+  },
+  "not_found": {
+    "title": "Oops!",
+    "go_home": "Vai alla schermata iniziale!",
+    "message": "Questa schermata non esiste."
+  },
+  "tabs": {
+    "messages": "Messaggi",
+    "search": "Cerca",
+    "events": "Eventi",
+    "home": "Home",
+    "communities": "Community"
+  },
+  "common": {
+    "cancel": "Cancella",
+    "add_photo": "Aggiungi foto",
+    "choose_photo_source": "Scegli da dove caricare la foto",
+    "take_photo": "Fai una foto",
+    "choose_from_library": "Scegli dalla galleria",
+    "open_settings": "Apri le impostazioni"
+  },
+  "update": {
+    "required_title": "Aggiornamento richiesto",
+    "body_warn": "Sembra che tu stia utilizzando una vecchia versione dell'app, quindi dovrai aggiornarla presto.",
+    "body_block": "Sembra che tu stia usando una vecchia versione dell'app, quindi devi aggiornarla adesso.",
+    "body_banned": "Sembra che la versione dell'app che stai utilizzando abbia dei bug, quindi dovrai aggiornarla.",
+    "preamble_warn": "Siamo un piccolo team di volontari e supportare le vecchie versioni è davvero difficile, ci scusiamo per il disagio.",
+    "preamble_block": "Siamo un piccolo team di volontari e supportare le vecchie versioni è davvero difficile, ci scusiamo per il disagio.",
+    "preamble_banned": "Siamo un piccolo team di volontari e a volte commettiamo degli errori, ci scusiamo per il disagio!",
+    "action_update": "Aggiorna ora",
+    "action_reinstall": "Scarica la nuova app",
+    "later": "Più tardi",
+    "failed": "L'aggiornamento non può essere applicato. Riprova per favore."
   }
 }