-
Notifications
You must be signed in to change notification settings - Fork 495
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
Support process applications (first iteration) #2508
Comments
Architecture DiscussionI (@philippfromme) discussed with @barmac. Key Points
File vs. Folder SupportFilesPros:
Cons:
Folders (reading BPMN, DMN, and Form files only)Pros:
Cons:
Next Steps
@nikku I would appreciate your input. |
I strongly support "folder reading support". We don't want to oversimplify our process applications feature, but support the full power of what pro-coders do:
|
Okay, we'll implement folders then. I'm also in favor of folders as I think with files only we'll end up wishing we'd implemented folders. 🤔 |
Is that because we create a pointer to each file, or for some other reason? In other words, is the process application only aware of the files in that folder, or does it track files even after they're moved out of the folder? |
The process application tracks any relevant files (BPMN, DMN and Form) in the root directory or any of its child directories. Moving a file out of that root directory will effectively remove the file from the process application. |
If the moved/deleted file is a linked resource, wouldn't that break the process application? Or is the first sentence above only referring to BPMN files? |
Of course it would break the process application, in similar ways like you fool someone by removing the chair he sits on: If the thing is used ("you sit on the chair") you'll notice that the thing ("the chair") is missing. If you're not sitting on it, then the operation is absolutely fine. We should build our validation in a robust manner, so that we detect that a piece we criticially rely on is now gone ("someone removed the chair I sit on"). This is out of question. But we should also build the underlying system in a way that it is robust, i.e. will be able to handle the situation where the chair you sit on is gone. (Not sure if that makes sense). |
That makes sense. Since "Deleting, renaming, or moving a file will not break the process application" was offered as a Pro of using folders, I'm trying to make sure I'm clear on what that means. |
Yes, we would show an error that the linked resource is now missing. 🙂 |
Supporting https://github.com/camunda/product-hub/issues/2458.
Introduction
Process applications are a feature of the Web Modeler that should be supported in the Desktop Modeler, too. Users should be able to work with and deploy process applications in the Desktop Modeler while still being able to work with and deploy single files at the same time.
Tasks
Out of scope
Related to https://github.com/bpmn-io/internal-docs/issues/519
The text was updated successfully, but these errors were encountered: