• Icon: Bug Report Bug Report
    • Resolution: Fixed
    • Icon: L3 - Default L3 - Default
    • 2.0.0
    • None
    • frontend
    • None

      Reproduce:

      • create a new report
      • change some data
      • save the report

      Expected:

      • report is saved with the new data

      Observed:

      • report has the old/initial data in it

        This is the controller panel for Smart Panels app

            [OPT-1118] Reports are not saved

            Same behavior appears to apply for Dashboards, too. Very strange.

            Sebastian Stamm added a comment - Same behavior appears to apply for Dashboards, too. Very strange.

            This appears to be a race condition: We are saving the Report/Dashboard and switching to view mode. This switch to view mode causes a re-instantiation of the Report/Dashboard component, loading the content. Depending on whether or not the backend already processed the saving, we get new or old content.

            Previously, a route change did not trigger a re-instantiation of the Report/Dashboard component, so that we did not need to query for the Report data from the backend. As we changed the way the Router resolves paths to components in OPT-1018, this bug was introduced.

            Sebastian Stamm added a comment - This appears to be a race condition: We are saving the Report/Dashboard and switching to view mode. This switch to view mode causes a re-instantiation of the Report/Dashboard component, loading the content. Depending on whether or not the backend already processed the saving, we get new or old content. Previously, a route change did not trigger a re-instantiation of the Report/Dashboard component, so that we did not need to query for the Report data from the backend. As we changed the way the Router resolves paths to components in OPT-1018 , this bug was introduced.

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

                Created:
                Updated:
                Resolved: