Azure / azure-functions-durable-js

JavaScript library for using the Durable Functions bindings
https://www.npmjs.com/package/durable-functions
MIT License
128 stars 47 forks source link

Should we rename the `callHttp()` asynchronousPatternEnabled property? #439

Closed hossam-nasr closed 1 year ago

hossam-nasr commented 1 year ago

This property defaults to true enables the polling behavior for context.callHttp() in orchestrations. The name sounds arcane though, but it's the name used by the durable extension. Should we use the v3 opportunity to rename it to a more descriptive name, e.g., enablePolling?

hossam-nasr commented 1 year ago

Discussed offline. Let's support this in a non-breaking way by adding a new enablePolling parameter, but keeping the old asynchronousPatternEnabled property with a @deprecated tag and a note to switch to the newer label

hossam-nasr commented 1 year ago

Closed by #460