Uploaded image for project: 'Camunda Optimize'
  1. Camunda Optimize
  2. OPT-1470 Move repeating http requests from IT to a separate module
  3. OPT-1479

Refactor IT for dashboards, reports and alerts rest endpoints

XMLWordPrintable

    • Icon: Sub-task Sub-task
    • Resolution: Fixed
    • Icon: L3 - Default L3 - Default
    • 2.3.0-alpha1, 2.3.0
    • None
    • backend, testing
    • None

      AT:

      • The IT classes related to dashboards, alerts and reports do not contain any rest calls, but refer to a helper-request methods in a separate module

        This is the controller panel for Smart Panels app

            [OPT-1479] Refactor IT for dashboards, reports and alerts rest endpoints

            Johannes added a comment -

            Looks really nice!

            Review hints:

            Johannes added a comment - Looks really nice! Review hints: for the first commit you messed up the ticket number and therefore it is not shown here. Link to the commit: https://github.com/camunda/camunda-optimize/commit/b45f3585a43a7204f2c2d017cb26ac454bfd41e5 OptimizeRequestExecutor: the clazz parameter in the execute and executeAndReturnList method have a different name, since this is not very expressive. I would rather called it classToExtractFromReponse or something similar

              Unassigned Unassigned
              kyrylo.zakurdaiev Kyrylo Zakurdaiev
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: