Remove the dead CultureMiddleware and centralize the supported-culture list
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 76/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- csharp
- Área
- backend, localization
Línea de trabajo
Empieza por LearnStack/Program.cs:85 y compara las listas de culturas en Middleware/CultureMiddleware.cs, Controllers/CultureController.cs y Extensions/LocalizationExtensions.cs. Elimina el middleware no utilizado y centraliza la lista mediante LocalizationExtensions. Después, verifica que el cambio de idioma mediante cookie y Accept-Language siga funcionando y que solo quede una lista de culturas compatibles.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Tier 3 — cleanup
Problem
CultureMiddleware implements full cookie + Accept-Language culture negotiation but is never registered — Program.cs uses UseRequestLocalization() instead, which does the same job. The dead middleware is a maintenance trap: the supported-culture list is now duplicated in three places that can silently drift apart.
Evidence
LearnStack/Middleware/CultureMiddleware.cs— never referenced inProgram.csLearnStack/Program.cs:85—app.UseRequestLocalization()- Duplicated culture lists:
Middleware/CultureMiddleware.cs:10,Controllers/CultureController.cs:10,Extensions/LocalizationExtensions.cs:7
Proposed fix
- Delete
CultureMiddleware.cs. - Expose one
SupportedCulturesconstant (inLocalizationExtensions) and reference it fromCultureController.
Acceptance criteria
- One source of truth for supported cultures
- Language switching still works via cookie and
Accept-Language
- Lenguaje dominante
- HTML
- Estrellas
- 10
- Forks
- 0
- Merge medio
- 2 h 35 min
- PR fusionados (30 d)
- 21
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de kasuken/LearnStack
-
enhancement
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
kasuken/LearnStack#59 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
kasuken/LearnStack#72 ·
Los mantenedores suelen responder en 1 día
-
Feature: admin dashboardAbiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
kasuken/LearnStack#71 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
kasuken/LearnStack#70 ·
Los mantenedores suelen responder en 1 día
-
Feature: public profile pagesAbiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 45/100
kasuken/LearnStack#69 ·
Los mantenedores suelen responder en 1 día
Todos los issues de kasuken/LearnStack
Issues similares
-
layer.setProps: opacity above 1 is clamped silently (layerStyle uses 0..100)Posiblemente ocupada @switch87 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
storytold/photocraft#5242 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
pinterest/pymemcache#662 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
-
[Enhancement]: Add Cheaper Inference to known custom endpoint providersPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto✨ enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
LibreChat-AI/LibreChat#17075 ·
Los mantenedores suelen responder en 1 día