admin - the redux store expose currentX we should drop that and use a pointer to the runs array

This issue is archived. You can view it, but you can't modify it. Learn more

XMLWordPrintable

      logically ... the runs in "currentRuns" must also exist under "runs". The currentRuns is more a helper for the view, but we should just use a pointer in the array as currentRuns selector and not a state variable.

      https://github.com/reactjs/reselect#accessing-react-props-in-selectors

      [ACTION_TYPES.GET_CURRENT_RUN_DATA](state, props,

      { payload }

      ): State

      { const currentRuns = state.get('runs'); return currentRuns[props.pipeline]; }

      ,

            Assignee:
            Unassigned
            Reporter:
            James Dumay
            Archiver:
            Jenkins Service Account

              Created:
              Updated:
              Resolved:
              Archived: