Don't set a field as required if it's read-only
@CyberneticX-Tech ya está trabajando en esto.
Desde el 8/9/2026.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
Línea de trabajo
Comienza en src/marshmallow_sqlalchemy/convert.py, línea 427, y sigue cómo nullable, los valores predeterminados y dump_only determinan el indicador required. Confirma que los campos dump-only, incluido el caso de relationship descrito en el issue, no se marquen como required; el issue estará completo cuando el schema generado refleje ese comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
This line shows that fields' required flag will only depend on whether they are nullable and have a default value:
But we're running into cases where certain fields (eg relations for which we want to enforce loading through the foreign key, but still want dumping the whole nested schema) are marked as required even though they are also dump_only.
Any reason for not including not dump_only as a condition for marking as required?
- Lenguaje dominante
- Python
- Estrellas
- 580
- Forks
- 105
- Merge medio
- 9 h 8 min
- PR fusionados (30 d)
- 3
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 marshmallow-code/marshmallow-sqlalchemy
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
Dependency DashboardAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 45/100
-
I suspect a breaking change related to "include_relationships=True" and schema.load() in version 1.4.1Posiblemente ocupada @Fire162 la tomó hace 36 días. Abiertohelp wanted
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
marshmallow-code/marshmallow-sqlalchemy#664 · 5 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
marshmallow-code/marshmallow-sqlalchemy#580 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Todos los issues de marshmallow-code/marshmallow-sqlalchemy
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
btclib-org/btclib#3062 ·
Los mantenedores suelen responder en 1 día
-
Run evals automatically in CIAbiertoarea:evals area:infra enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
jse-avl/support-bot#4 · 1 comentario ·
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
qte77/agentic-job-offer-to-application-kit#487 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
On a free-threaded build, `_decrypted` can report a ledger locked mid-sign as a damaged fileAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
btclib-org/btclib-wallet#801 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
documentation good first issue
Dificultad 1/5 1-3 horas Aptitud para principiantes 94/100