Merge pull request #672 from madeofpendletonwool/translation-system

Translation system

Author: madeofpendletonwool

.github/workflows/build-helm-chart.yml

@@ -26,13 +26,8 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
         with:
-          persist-credentials: false # This prevents the default token from being persisted in the local git config
-
-      - name: Setup Git for push
-        run: |
-          git config --global user.name "github-actions[bot]"
-          git config --global user.email "github-actions[bot]@users.noreply.github.com"
-          git remote set-url origin https://x-access-token:${{ secrets.PUSH_PAT }}@github.com/${{ github.repository }}.git
+          token: ${{ secrets.PUSH_PAT }}
+          persist-credentials: true
 
       - name: Setup Helm
         uses: Azure/setup-helm@v4.2.0
@@ -84,4 +79,3 @@ jobs:
           committer_email: actions@github.com
           message: "Update Helm chart for release ${{ github.event.release.tag_name }}"
           add: "docs"
-          push: "origin main"

database_functions/migration_definitions.py

@@ -2563,6 +2563,59 @@ def migration_029_fix_people_episodes_table_schema(conn, db_type: str):
         cursor.close()
 
 
+@register_migration("030", "add_user_language_preference", "Add Language column to Users table for user-specific language preferences", requires=["001"])
+def migration_030_add_user_language_preference(conn, db_type: str):
+    """Add Language column to Users table for user-specific language preferences"""
+    cursor = conn.cursor()
+    
+    try:
+        # Get the default language from environment variable, fallback to 'en'
+        default_language = os.environ.get("DEFAULT_LANGUAGE", "en")
+        
+        # Validate language code (basic validation)
+        if not default_language or len(default_language) > 10:
+            default_language = "en"
+            
+        logger.info(f"Adding Language column to Users table with default '{default_language}'")
+        
+        if db_type == 'postgresql':
+            # Add Language column with default from environment variable
+            safe_execute_sql(cursor, f'''
+                ALTER TABLE "Users" 
+                ADD COLUMN IF NOT EXISTS Language VARCHAR(10) DEFAULT '{default_language}'
+            ''', conn=conn)
+            
+            # Add comment to document the column
+            safe_execute_sql(cursor, '''
+                COMMENT ON COLUMN "Users".Language IS 'ISO 639-1 language code for user interface language preference'
+            ''', conn=conn)
+            
+        else:  # mysql/mariadb
+            # Check if column exists first
+            cursor.execute("""
+                SELECT COUNT(*) 
+                FROM INFORMATION_SCHEMA.COLUMNS 
+                WHERE TABLE_SCHEMA = DATABASE() 
+                AND TABLE_NAME = 'Users' 
+                AND COLUMN_NAME = 'Language'
+            """)
+            
+            if cursor.fetchone()[0] == 0:
+                safe_execute_sql(cursor, f'''
+                    ALTER TABLE Users 
+                    ADD COLUMN Language VARCHAR(10) DEFAULT '{default_language}' 
+                    COMMENT 'ISO 639-1 language code for user interface language preference'
+                ''', conn=conn)
+        
+        logger.info(f"Successfully added Language column to Users table with default '{default_language}'")
+
+    except Exception as e:
+        logger.error(f"Error in migration 030: {e}")
+        raise
+    finally:
+        cursor.close()
+
+
 # ============================================================================
 # GPODDER SYNC MIGRATIONS
 # These migrations match the gpodder-api service migrations from Go code
@@ -3043,14 +3096,77 @@ def migration_105_optimize_episode_actions_performance(conn, db_type: str):
             ''', conn=conn)
 
         logger.info("Successfully added episode actions performance indexes")
-        
+
     except Exception as e:
         logger.error(f"Error in gpodder migration 105: {e}")
         raise
     finally:
         cursor.close()
 
 
+@register_migration("106", "optimize_subscription_sync_performance", "Add missing indexes for subscription sync queries", requires=["103"])
+def migration_106_optimize_subscription_sync_performance(conn, db_type: str):
+    """Add critical indexes for subscription sync performance to prevent AntennaPod timeouts"""
+    cursor = conn.cursor()
+
+    try:
+        logger.info("Adding performance indexes for subscription sync...")
+
+        if db_type == 'postgresql':
+            # Critical indexes for subscription sync performance
+            safe_execute_sql(cursor, '''
+                CREATE INDEX IF NOT EXISTS idx_gpodder_sync_subs_user_device_timestamp
+                ON "GpodderSyncSubscriptions"(UserID, DeviceID, Timestamp DESC)
+            ''', conn=conn)
+
+            safe_execute_sql(cursor, '''
+                CREATE INDEX IF NOT EXISTS idx_gpodder_sync_subs_user_action_timestamp
+                ON "GpodderSyncSubscriptions"(UserID, Action, Timestamp DESC)
+            ''', conn=conn)
+
+            safe_execute_sql(cursor, '''
+                CREATE INDEX IF NOT EXISTS idx_gpodder_sync_subs_podcast_url_user
+                ON "GpodderSyncSubscriptions"(UserID, PodcastURL, Timestamp DESC)
+            ''', conn=conn)
+
+            # Optimize subscription change queries with compound index
+            safe_execute_sql(cursor, '''
+                CREATE INDEX IF NOT EXISTS idx_gpodder_sync_subs_complex_query
+                ON "GpodderSyncSubscriptions"(UserID, DeviceID, Action, Timestamp DESC, PodcastURL)
+            ''', conn=conn)
+
+        else:  # mysql/mariadb
+            # Critical indexes for subscription sync performance
+            safe_execute_sql(cursor, '''
+                CREATE INDEX idx_gpodder_sync_subs_user_device_timestamp
+                ON GpodderSyncSubscriptions(UserID, DeviceID, Timestamp DESC)
+            ''', conn=conn)
+
+            safe_execute_sql(cursor, '''
+                CREATE INDEX idx_gpodder_sync_subs_user_action_timestamp
+                ON GpodderSyncSubscriptions(UserID, Action, Timestamp DESC)
+            ''', conn=conn)
+
+            safe_execute_sql(cursor, '''
+                CREATE INDEX idx_gpodder_sync_subs_podcast_url_user
+                ON GpodderSyncSubscriptions(UserID, PodcastURL(255), Timestamp DESC)
+            ''', conn=conn)
+
+            # Optimize subscription change queries with compound index
+            safe_execute_sql(cursor, '''
+                CREATE INDEX idx_gpodder_sync_subs_complex_query
+                ON GpodderSyncSubscriptions(UserID, DeviceID, Action, Timestamp DESC, PodcastURL(255))
+            ''', conn=conn)
+
+        logger.info("Successfully added subscription sync performance indexes")
+
+    except Exception as e:
+        logger.error(f"Error in gpodder migration 106: {e}")
+        raise
+    finally:
+        cursor.close()
+
+
 if __name__ == "__main__":
     # Register all migrations and run them
     register_all_migrations()

deployment/docker/compose-files/docker-compose-mysql/docker-compose.yml

@@ -42,6 +42,11 @@ services:
       PGID: ${GID:-911}
       # Add timezone configuration
       TZ: "America/New_York"
+      # Language Configuration
+      DEFAULT_LANGUAGE: "en"
+    volumes:
+      # Mount the download and the backup location on the server if you want to. You could mount a nas to the downloads folder or something like that.
+      # The backups directory is used if backups are made on the web version on pinepods. When taking backups on the client version it downloads them locally.
 
     volumes:
       # Mount the download and backup locations on the server

deployment/docker/compose-files/docker-compose-postgres/docker-compose.yml

@@ -40,6 +40,8 @@ services:
       PGID: ${GID:-911}
       # Add timezone configuration
       TZ: "America/New_York"
+      # Language Configuration
+      DEFAULT_LANGUAGE: "en"
     volumes:
       # Mount the download and backup locations on the server
       - /home/user/pinepods/downloads:/opt/pinepods/downloads

dockerfile

@@ -117,6 +117,8 @@ ENV TZ=UTC
 COPY --from=python-builder /build/dist/pinepods-db-setup /usr/local/bin/
 # Copy built files from the builder stage to the Nginx serving directory
 COPY --from=builder /app/dist /var/www/html/
+# Copy translation files for the Rust API to access
+COPY ./web/src/translations /var/www/html/static/translations
 # Copy Go API binary from the go-builder stage
 COPY --from=go-builder /gpodder-api/gpodder-api /usr/local/bin/
 # Copy Rust API binary from the rust-api-builder stage

dockerfile-arm

@@ -141,6 +141,8 @@ ENV TZ=UTC
 COPY --from=python-builder /build/dist/pinepods-db-setup /usr/local/bin/
 # Copy built files from the builder stage to the Nginx serving directory
 COPY --from=builder /app/dist /var/www/html/
+# Copy translation files for the Rust API to access
+COPY ./web/src/translations /var/www/html/static/translations
 # Copy Go API binary from the go-builder stage
 COPY --from=go-builder /gpodder-api/gpodder-api /usr/local/bin/
 # Copy Rust API binary from the rust-api-builder stage