Skip to content

HashSerializer#1

Open
KumatomatomaKu wants to merge 1 commit into0.10.5_basefrom
0.10.5_with_hash_serializer
Open

HashSerializer#1
KumatomatomaKu wants to merge 1 commit into0.10.5_basefrom
0.10.5_with_hash_serializer

Conversation

@KumatomatomaKu
Copy link
Owner

@KumatomatomaKu KumatomatomaKu commented May 26, 2017

  • Invisible to serializer_for method
    • One needs to new HashSerializer every time
    • The main reason is that Collection => Hash won't work correctly for now and cause tests to fail
  • Set option uniform_collection: true so that HashSerializer check array values' first element only and assume that all the rest should use then same serializer
  • int or string values work fine, no exception will be thrown
    • But be ware that values without suitable serializers will also be accepted and cause maybe unexpected output

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