Skip to content

Support the new Node.js programming model in durable #384

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ejizba opened this issue Sep 2, 2022 · 1 comment · May be fixed by Arcuity-ai/azure-functions-durable-js#15
Closed

Support the new Node.js programming model in durable #384

ejizba opened this issue Sep 2, 2022 · 1 comment · May be fixed by Arcuity-ai/azure-functions-durable-js#15

Comments

@ejizba
Copy link
Contributor

ejizba commented Sep 2, 2022

More info here: https://aka.ms/AzFuncNodeV4

Here are my recommended steps to do this in an incremental fashion:

  1. This will be a breaking change, there's no way around that. So to start, I'd create a new branch, maybe "v3.x" and update the version of this package to "3.0.0-alpha.1"
  2. I would update the reference of "@azure/functions" in the package.json to "4.0.0-alpha.2"
  3. I would fix all the build errors (there will be a lot lol) and submit a PR
  4. Now, I would try to get it to actually run locally 😅 That means following steps here and using app.generic to register durable functions. Once it works locally, I'd submit another PR as I doubt the changes from step 3 will work on the first attempt
  5. Finally, I would add fancy intellisense for app.durable methods. Currently tracked by this issue, but open for discussion where that code lands. Regardless, ideally this is a separate PR
@hossam-nasr
Copy link
Contributor

First alpha v3 release of the SDK with programming model support is out now! 🎉

For further work on the programming model, discussions and improvements, closing this in favor of #393, #394, #414, #415, #416, #417, #418, #419, #420, #421

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants