PipedreamHQ / roadmap

Pipedream is an integration platform for developers. Have a feature request or bug to report? Add it here.
https://pipedream.com
30 stars 5 forks source link

[FEATURE] Add workflow name to original_context #416

Closed cfjedimaster closed 4 years ago

cfjedimaster commented 4 years ago

Is your feature request related to a problem? Please describe. The original_context object passed to the error workflow has the workflow id that caused the issue. It should also contain the name of the workflow. I'm assuming there is some way to get this name via code, but it should just be passed in.

Describe the solution you'd like Add workflow_name to original_context.

Do you have a workaround? No. I assume there's a way to do it though. :)

dylburger commented 4 years ago

@cfjedimaster thanks for the request! We're tracking this at #412 .

cfjedimaster commented 4 years ago

One thing though - that issue relates to the email generated by the workflow. That is a good idea and should be done, but for folks who do their own error handling, or modify the workflow etc, it would be helpful to have this value. Basically I'm saying if you add some code to the error workflow to fix #412, I'd still want this particular feature done too. :)

dylburger commented 4 years ago

@cfjedimaster yes I understand the distinction - we'll likely solve this by including the name in the original_context, or perhaps allowing you to access metadata on your own workflows via API.

We mainly just try to consolidate issues we know will be tackled by the same work and reference them in a single ticket. I also updated that ticket name to reflect your use case.