Skip to content

Ignore field metadata with non-string keys#596

Merged
lafrech merged 1 commit intodevfrom
non_string_metadata_keys
Aug 29, 2020
Merged

Ignore field metadata with non-string keys#596
lafrech merged 1 commit intodevfrom
non_string_metadata_keys

Conversation

@lafrech
Copy link
Copy Markdown
Member

@lafrech lafrech commented Aug 28, 2020

Rebase / squash of #585.

(I couldn't find a way to push in PR branch.)

Closes #585.

@lafrech lafrech changed the title Support field metadata with non-string keys Ignore field metadata with non-string keys Aug 29, 2020
@lafrech lafrech force-pushed the non_string_metadata_keys branch from 67be224 to e4704b8 Compare August 29, 2020 20:12
@lafrech
Copy link
Copy Markdown
Member Author

lafrech commented Aug 29, 2020

I'm not sure marshmallow really means to authorize non-string keys in metadata, but it doesn't hurt to add this check.

@lafrech lafrech merged commit 27d8dd5 into dev Aug 29, 2020
@lafrech lafrech deleted the non_string_metadata_keys branch August 29, 2020 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant