temporalio / sdk-ruby

Temporal Ruby SDK
MIT License
74 stars 7 forks source link

[Fix] DataConverter improvements #80

Closed antstorm closed 1 year ago

antstorm commented 1 year ago

What was changed

This PR modifies Client and Worker interfaces to accept a single DataConverter as an argument instead of individual pieces(payload converter, codecs and failure converter).

It also includes a bug fix for the DataConverter#to_payload and DataConverter#from_payload to use codecs.

Why?

This is one of the follow-ups to #74

Checklist

  1. Closes

    75

    77

  2. How was this tested: Specs and RBS updated

  3. Any docs updates needed? Inline documentation has been updated