Skip to content

[ElementsPM] integrate Elements ProcessManager#435

Open
dpfaffenbauer wants to merge 2 commits into
instride-ch:5.0from
dpfaffenbauer:elements-pm
Open

[ElementsPM] integrate Elements ProcessManager#435
dpfaffenbauer wants to merge 2 commits into
instride-ch:5.0from
dpfaffenbauer:elements-pm

Conversation

@dpfaffenbauer

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread src/DataDefinitionsBundle/Command/ImportCommand.php Outdated
Co-authored-by: dolmit-tpaaro <117173069+dolmit-tpaaro@users.noreply.github.com>
@dolmit-tpaaro

dolmit-tpaaro commented Jul 31, 2025

Copy link
Copy Markdown

@dpfaffenbauer Hi,

Is this PR a follow up to this: #434?
To be honest, I was rather surprised that integration with your https://github.com/dpfaffenbauer/ProcessManager bundle was removed. Mainly because there's no information about why it was removed. Would've been great to have some kind of information that one bundle is going to be replaced by the other. :)

@dpfaffenbauer

Copy link
Copy Markdown
Collaborator Author

@dolmit-tpaaro it should be quite obvious, https://github.com/dpfaffenbauer/ProcessManager hasn't gotten any updates for 2 years

@dolmit-tpaaro

dolmit-tpaaro commented Aug 1, 2025

Copy link
Copy Markdown

@dpfaffenbauer Yeah, I noticed the lack of updates.
What I'm trying to say is that any kind of heads-up about the switch would've been greatly appreciated. That's because we could've already start developing our own solution to this problem. That would've been wasted time.

@dpfaffenbauer

Copy link
Copy Markdown
Collaborator Author

@dolmit-tpaaro sorry mate. welcome to open-source, thats a hard pill to swallow. Projects do die more often than you think. You can still fork my process-manager and further maintain it.

@dolmit-tpaaro

dolmit-tpaaro commented Aug 1, 2025

Copy link
Copy Markdown

@dpfaffenbauer It's fine, though, you're misunderstanding me. I'm not against the switch nor want I maintain the deprecated bundle.
My only issue with the switch is that it was not communicated. The integration with your ProcessManagerBundle was dropped without any comment, leaving everybody in the dark.
In the future, please, communicate these major changes and further plans to the community.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants