Temporal.io support

[vsychov]

Hi there,

roadrunner 2+ have native support of Temporal.io, do you have any plans to implement support of this?
If not - will you accept this feature as PR to this bundle?

[Baldinof]

Hi!

As it's included in RoadRunner I would totally accept a PR :)

[xepozz]

Hi there!
I did some integration based on your bundle.
I will try to make PR during this week.

[xepozz]

I did demo application https://github.com/xepozz/symfony-temporal-demo and PR #49

[Kaspiman]

Let's take a look at this bundle https://github.com/VantaFinance/temporal-bundle

[praswicaksono]

@Baldinof @vsychov already make PR for it also #147

Any review would be appreciated

[Nyholm]

I love Temporal as much as the next person.

However, this bundle is great and the feature set has a limited scope. Would it make more sense to add a TemporalBundle that takes advances of this bundle?

I think that would make maintenance simpler and it will be less code for the people that is not using Temporal.

[xepozz]

@Nyholm how you'd use Temporal without RoadRunner?

[Baldinof]

To be honest, I would be willing to merge #147 but I don't have the time to do an actual review and test it.

The key thing is it should not add things as dependencies, and only enable the integration if it detects temporal.

[Nyholm]

I use this bundle for RoadRunner, but for my temporal integration, I have a private package. It sure does include some opinionated decisions which is why it not open sourced.

What I am suggesting:

1. For RoadRunner only: Use this bundle
2. For Temporal users: Use this bundle + a "Temporal specific bundle".

The "Temporal specific bundle" should depend on this RoadRunner bundle and use its extension points.

I asked to rebase and fix up #147 so it will be easier to review.