Uploaded image for project: 'Camunda Optimize'
  1. Camunda Optimize
  2. OPT-1400 I can combine reports
  3. OPT-1406

I can perform CRUD operations on combined reports using Rest-API

XMLWordPrintable

    • Icon: Feature Part Feature Part
    • Resolution: Fixed
    • Icon: L3 - Default L3 - Default
    • 2.2.0
    • None
    • backend
    • None

      AT:

      • I can perform CRUD (create/retrieve/update/delete) operations on combined reports using Rest-API
      • The CRUD operations still work for single reports
      • When fetching the list of all available reports, I also retrieve the combined reports
      • I can evaluate combined reports by id
      • I can evaluate unsaved combined reports
      • The documentation is up-to-date
      • An evaluation of an empty report returns an empty result

        This is the controller panel for Smart Panels app

            [OPT-1406] I can perform CRUD operations on combined reports using Rest-API

            Kyrylo Zakurdaiev added a comment -

            Review hints:

            • We could completely divide the endpoints for combined and single reports also for retrieving, updating and deleting those. That would better reflect how our database looks like and we won't need to save the redundant reportType property.
            • ReportRestServiceIT.java => asserting status code 500 while comments say “status code is OK”

            Kyrylo Zakurdaiev added a comment - Review hints: We could completely divide the endpoints for combined and single reports also for retrieving, updating and deleting those. That would better reflect how our database looks like and we won't need to save the redundant reportType property. ReportRestServiceIT.java => asserting status code 500 while comments say “status code is OK”

            Johannes added a comment -
            • Yes, you are definitely right - we should refactor that. I created OPT-1440 to tackle this issue
            • I removed the comments

            Johannes added a comment - Yes, you are definitely right - we should refactor that. I created OPT-1440 to tackle this issue I removed the comments

              Unassigned Unassigned
              johannes.heinemann Johannes
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: