https://github.com/UltimateAngular/redux-store/blob/9109ef8fd4b0639008975682fc4e7be33b275a1f/src/store/actions.ts#L1-L3

I think it's not an issue, but a small addition. For my point of view the enum will look more consistent for action names constants.

export enum ActionTypes {
	ADD_TODO = '[Todo] Add todo',
	REMOVE_TODO = '[Todo] Remove todo'
}

https://github.com/UltimateAngular/redux-store/blob/9109ef8fd4b0639008975682fc4e7be33b275a1f/src/app.ts#L39

Why don't to use a dataset property of HTMLElement instead of getAttribute?

const payload = JSON.parse(target.dataset.todo as any);

I got this error from Webpack:

Seemed to be resolved by switching to ts-loader and updating which version of Webpack the project is using to 4.12.0:

webpack.config.js:

module.exports = {
  entry: './src/app.ts',
  output: {
    filename: 'app.js',
    path: __dirname + '/dist/',
  },
  devServer: {
    port: 8000,
  },
  resolve: {
    extensions: ['.ts', '.js'],
  },
  module: {
    rules: [
      {
        test: /\.ts$/,
        use: 'ts-loader',
      },
    ],
  },
};

package.json:

{
  "name": "store",
  "version": "0.0.0",
  "license": "MIT",
  "scripts": {
    "start": "webpack-dev-server"
  },
  "devDependencies": {
    "awesome-typescript-loader": "3.2.3",
    "ts-loader": "^4.4.1",
    "typescript": "2.5.3",
    "webpack": "4.12.0",
    "webpack-cli": "^3.0.8",
    "webpack-dev-server": "3.1.4"
  }
}

Is there another way in which to resolve this issue? @toddmotto

First, I want to thank you very much. I learnt a lot!

I have a comment:
I think getting the value of the state like that still exposes it to mutation (because objects are assigned by reference)
https://github.com/UltimateAngular/redux-store/blob/9109ef8fd4b0639008975682fc4e7be33b275a1f/src/store/store.ts#L13

consider if I add this to button.addEventListener

    let state = store.value;
    state.todos = { mutated: true };
    console.log('Original state:::', store.value);
    console.log('Mutated state:::', state);

I think it is better to do this

  get value() {
    return { ...this.state };
  }

please correct me if I'm wrong. I'm still learning :)

thanks 👍

https://github.com/UltimateAngular/redux-store/blob/9109ef8fd4b0639008975682fc4e7be33b275a1f/src/store/reducers.ts#L35

And again a small addition. Maybe it's more consistent to stay inside our switch...case structure and return state in default section instead of function's return?!

default:
    return state;