type-safe persistStore()

[arzyu]

After this patch, we can persistStore(store) without any type errors when using store: Store<RootState, RootAction>.

Here is my usage scenario:

import React, { StatelessComponent } from "react";
import { Provider } from "react-redux";
import { Route } from "react-router-dom";
import { Store } from "redux";
import { persistStore } from "redux-persist";
import { PersistGate } from "redux-persist/integration/react";

import { Router } from "./Router";
import { RootState, RootAction } from "./store";

interface Props {
  store: Store<RootState, RootAction>;
  app: StatelessComponent;
}

export const Root: StatelessComponent<Props> = ({ store, app }) => (
  <Provider store={store}>
    <PersistGate persistor={persistStore(store)}>
      <Router>
        <Route path="/:path?" component={app} />
      </Router>
    </PersistGate>
  </Provider>
);

[javierbustamante]

Someone knows what happen with the PR's in this repo?

[rajharshwal92]

Kindly merge this Branch. This PR can solve type error related to store.

[daruj]

Hey guys, when are u merging this?

[MichelMelhem]

up

[JuanGilArizon]

I think ive solved it just adding "PersistPartial" type in the createStore function: const store = createStore<IRootState & PersistPartial, any, any, any>( persistedReducer ); Hope it helps somebody

[guitoof]

Hi there, this looks like a decent solution to me! Thank you very much, just patched it in my project 🙏 Do you guys think it can be merged?

[nicoleCamoro]

This looks like it could fix my problem. Is this gonna get merged?

[ahmadichsan]

Hello, any update for this PR? Seems like a solution for type error issue related to store.

[BenoitSerrano]

Hello ! I patched it into my project and it works :) Could you merge it?

[artdevgame]

@rt2zz Is it possible to merge this in please?

[gpawlik]

@sunnylqm Is there any reason it cannot be merged yet?

[ease-space]

Why cannot be merge???

[sunnylqm]

I do not have the permission. I made an approval to try to remind those who have permissions to merge. (Yes, anyone can approve)

[gpawlik]

Thanks for the message @sunnylqm . I guess the question should be directed rather to @rt2zz then.

[ease-space]

@rt2zz why cannot be merge?

[NoahFiner]

can we merge? :)

[ronaldruzicka]

@rt2zz please merge, I need this as well 🙏

[Jonathan0wh]

any update on when it will be merged?

[mfrey777]

Is there any news on the merge of this PR, would be very much appreciated.

[lawhorkl]

waiting over a year for this to get merged, this is kind of ridiculous

[Jonathan0wh]

It's still not merged?

[Rocamonde]

Kindly merge this and include it in a new release as soon as possible.

[delewis13]

@arzyu :)

[arzyu]

I don’t have permission to merge code, hope the maintainer will continue to be active.

[irinrudko]

I love it! Thank you so much! It's the only source where I could've found this.

[malininss]

please merge this

[anaerobic]

has conflicts @arzyu are you able to resolve?