Quick fix for setting fulfillmentStatus

intacct / intacct-sdk-js

Official repository of the Sage Intacct SDK for JavaScript in Node.js

https://developer.intacct.com/tools/sdk-node-js/

Apache License 2.0

23 stars 33 forks source link

Quick fix for setting fulfillmentStatus #78

Closed lprichar closed 2 years ago

lprichar commented 2 years ago

A more full featured fix would allow setting deliverydate, deferralstatus, and kitstatus, however that would be a breaking change because the data type for fulfillmentstatus would change from a string to a class. This implementation just fixes the broken code by setting fulfillmentstatus into deliverystatus, which is probably what was intended to begin with.

lprichar commented 2 years ago

This addresses Bug #77

dylan-holt-sage commented 2 years ago

@lprichar, looks good, merging to upcoming release version for testing purposes. Thanks for your contribution :)