have redux-saga removed

At the moment we are using redux-saga as part of the state-management. This shouldn’t be needed since the data-flow within the dext itself is pretty straight forward.

we can even go as far as to consider having redux removed.

There are some strong points where redux would shine in a smaller application like dext but they wouldn’t outweigh simplicity nor attempt to just use React’s own recommended patterns.

---

something to consider

In fact, IMO using the new Context API would be the perfect tool for us to leverage for our use case, such as UI configuration, in conjunction with just props being passed down from one “interface” or prop-getters.

[adnasa]

One cool thing about using context here would be to eliminate imports of ipcRenderer, which essentially means that we leak electron functionality into our components.

Admittedly, I have yet to try out the new Context API, but even with the old approach, you could see the benefit

class A extends React.Component {
  getChildContext () {
    return {
      socket: ipcRenderer,
    }
  }
}

class B extends React.Component {
  static contextTypes = {
    // because `context`, `socket` doesn't have to interface `ipcRenderer`
    socket: PropTypes.shape({
      on: PropTypes.function.isRequired,
    })
  }
}

// unit-test
// mock all the things..
const view = shallow(
  <B {...props} />,
  {
    socket: {
      on: jest.fn()
    }
  }
)

[adnasa]

Let’s try to get React updated to the latest version

I’m on it 😄