marshmallow-code/marshmallow

Allow specifying output field when using `unknown=INCLUDE`

Offen

#853 geöffnet am 23.06.2018

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (4 Reaktionen) (0 zugewiesene Personen)Python (640 Forks)batch import
feedback welcomehelp wanted

Repository-Metriken

Stars
 (6.787 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 10h 55m) (7 gemergte PRs in 30 T)

Beschreibung

From https://github.com/marshmallow-code/marshmallow/issues/524#issuecomment-393057138

We could even go further and let the user set a default Field when using INCLUDE, but I'm not sure it is so useful, and it is never too late to add it in a later step.

(edited to make the argument name correct)

I suggested here that a field could be passed to unknown, e.g. unknown=fields.Str().

Contributor Guide

Research-Richtung
Untersuche, wie die Serialisierungslogik geändert werden kann, um einen Feldparameter zu akzeptieren, wenn `unknown=INCLUDE` gesetzt ist. Sieh dir die aktuelle Behandlung unbekannter Felder an und schlage eine Möglichkeit vor, eine Feldinstanz zur Serialisierung unbekannter Felder zu übergeben.
Tech Stack
python
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Pythonmarshmallow
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70