Add (Un)Marshal Text/Binary methods to support json parsing #670

aslakknutsen · 2016-08-01T11:49:52Z

This is required by Client to parse Payload with UUID in it.

aslakknutsen · 2016-08-01T11:57:18Z

No idea how this effect the GopherJS client.

encoding/json/encoding_suite_test.go

+	"testing"
+)
+
+func TestMiddleware(t *testing.T) {


uuid/uuid.go

+
+// UnmarshalBinary Wrapper over the real UnmarshalBinary method
+func (u *UUID) UnmarshalBinary(data []byte) error {
+	t := uuid.UUID{}


) * Add (Un)Marshal Text/Binary methods to support json parsing * Add tests and fix impl ;) * Add local test setup. Add round trip test

Add (Un)Marshal Text/Binary methods to support json parsing

1e37c71

Add tests and fix impl ;)

b4150f7

raphael reviewed Aug 1, 2016
View reviewed changes

encoding/json/encoding_suite_test.go

"testing"

)

func TestMiddleware(t *testing.T) {

This comment was marked as off-topic.

Sign in to view

This comment was marked as off-topic.

Sign in to view

Add local test setup. Add round trip test

6b689c2

raphael reviewed Aug 2, 2016
View reviewed changes

raphael merged commit db7bfc2 into goadesign:master Aug 2, 2016

raphael pushed a commit that referenced this pull request Aug 3, 2016

Add (Un)Marshal Text/Binary methods to support json parsing (#670) (#678

6c2c418

) * Add (Un)Marshal Text/Binary methods to support json parsing * Add tests and fix impl ;) * Add local test setup. Add round trip test

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add (Un)Marshal Text/Binary methods to support json parsing #670

Add (Un)Marshal Text/Binary methods to support json parsing #670

aslakknutsen commented Aug 1, 2016

aslakknutsen commented Aug 1, 2016

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

Add (Un)Marshal Text/Binary methods to support json parsing #670

Add (Un)Marshal Text/Binary methods to support json parsing #670

Conversation

aslakknutsen commented Aug 1, 2016

aslakknutsen commented Aug 1, 2016

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.