[Feature Request] Specify activity names at runtime

Is your feature request related to a problem? Please describe.

I have a need to create activities with names that aren’t known until runtime. The current design requires me to use [Activity(string)] on a method or delegate, which only allows me to use activity names known at compile time.

As far as I can tell, there’s no simple way for me to add an ActivityAttribute to a Delegate at runtime in .NET, short of doing some kind of dynamic type generation (which seems like overkill).

Describe the solution you’d like

A simple solution would be to add a TemporalWorkerOptions.AddActivity(string name, Delegate del) overload that allows me to specify the activity name explicitly.

[cretz]

@cgillum - I have already started on your first suggestion. I am right in the middle of workflow development on https://github.com/cretz/temporal-sdk-dotnet/tree/workflow-more-impl but I have exposed activity definition publicly already. But that code is a mess because it is mid development. Give me a little while longer and another alpha will be released with full workflow support (due to vacation and such, this may not be at least until the week of the 20th).

Also in the meantime, would love feedback on SDK approach/code in general. Though not quite yet on that branch 😃 May want to wait until the next workflow PR is up hopefully this week (will be merging #29 before I open the next one since I’m daisy-chaining for feedback reasons).

As for full impl of #27 and dynamic activity support, I need to wait until after I finish this development on workflows.

[cgillum]

Yeah, this is effectively a general-purpose library for others to use, so the single activity approach unfortunately won’t work for me.

Thanks for opening the other issue - I’ll keep an eye on that.