Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-6476

Update isScanForProcessDefinitions property description in processes.xml documentation

      The isScanForProcessDefinitions property does not only include *.bpmn20.xml and *.bpmn file suffixes, all supported suffixes are deployed. The documentation needs to be updated to reflect this.

      Docs: https://docs.camunda.org/manual/7.5/user-guide/process-applications/the-processes-xml-deployment-descriptor/

      Code: https://github.com/camunda/camunda-bpm-platform/blob/master/engine/src/main/java/org/camunda/bpm/container/impl/deployment/scanning/ProcessApplicationScanningUtil.java#L68

        This is the controller panel for Smart Panels app

            [CAM-6476] Update isScanForProcessDefinitions property description in processes.xml documentation

            Matthijs created issue -
            Matthijs made changes -
            Status Original: Open [ 1 ] New: In Progress [ 3 ]
            Matthijs made changes -
            Assignee New: Matthijs [ matthijs.burke ]
            Matthijs made changes -
            Fix Version/s New: 7.7.0 [ 14607 ]
            Fix Version/s New: 7.6.0 [ 14490 ]
            Fix Version/s New: 7.5.0 [ 14091 ]
            Fix Version/s New: 7.4.0 [ 13505 ]

            Matthijs added a comment -

            thorben.lindhauer, can you please do this review?

            Thanks and brgds

            Matthijs added a comment - thorben.lindhauer , can you please do this review? Thanks and brgds
            Matthijs made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
            Remaining Estimate New: 0 minutes [ 0 ]
            Original Estimate New: 0 minutes [ 0 ]
            Matthijs made changes -
            Assignee Original: Matthijs [ matthijs.burke ] New: Thorben Lindhauer [ thorben.lindhauer ]

            Hi Mat,

            I think we should update the process-archive reference accordingly: https://docs.camunda.org/manual/7.5/reference/deployment-descriptors/tags/process-archive/#configuration-properties. In the description of isScanForProcessDefinitions, we should mention that also resources matching the additionalResourceSuffixes parameter are included in the scan.

            Cheers,
            Thorben

            Thorben Lindhauer added a comment - Hi Mat, I think we should update the process-archive reference accordingly: https://docs.camunda.org/manual/7.5/reference/deployment-descriptors/tags/process-archive/#configuration-properties . In the description of isScanForProcessDefinitions , we should mention that also resources matching the additionalResourceSuffixes parameter are included in the scan. Cheers, Thorben
            Thorben Lindhauer made changes -
            Assignee Original: Thorben Lindhauer [ thorben.lindhauer ] New: Matthijs [ matthijs.burke ]
            Resolution Original: Fixed [ 1 ]
            Status Original: Resolved [ 5 ] New: Reopened [ 4 ]
            Matthijs made changes -
            Assignee Original: Matthijs [ matthijs.burke ] New: Thorben Lindhauer [ thorben.lindhauer ]
            Resolution New: Fixed [ 1 ]
            Status Original: Reopened [ 4 ] New: Resolved [ 5 ]

              thorben.lindhauer Thorben Lindhauer
              matthijs.burke Matthijs
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: