Skip to content
This repository has been archived by the owner on May 18, 2020. It is now read-only.

New npm module #1

Merged
merged 2 commits into from
Feb 21, 2018
Merged

New npm module #1

merged 2 commits into from
Feb 21, 2018

Conversation

jamescoq
Copy link
Collaborator

No description provided.

.editorconfig Outdated
@@ -0,0 +1,11 @@
# http://editorconfig.org
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sjednotit s react-union

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

README.md Outdated
@@ -1,3 +1,104 @@
# redux-extensible-store

Redux extension that allows you to dynamickly inject new reducers and sagas into you running application.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dynamically

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

README.md Outdated

## Why?

If you are using dynamic imports or some kind of a code splitting then you probably need to register new reducers and sagas the time your JS chunk is loaded. So what does this module provides? We call these ad-hoc loaded modules a widgets.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

at the time

@@ -0,0 +1,26 @@
import babel from 'rollup-plugin-babel';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mrkni na rollup.config v react-union, generuje se tam ještě ecmascript build

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

.editorconfig Outdated
@@ -0,0 +1,11 @@
# http://editorconfig.org
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

prosím, sjednotit s react-union

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

README.md Outdated
@@ -1,3 +1,104 @@
# redux-extensible-store

Redux extension that allows you to dynamickly inject new reducers and sagas into you running application.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dynamically

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

README.md Outdated

## Why?

If you are using dynamic imports or some kind of a code splitting then you probably need to register new reducers and sagas the time your JS chunk is loaded. So what does this module provides? We call these ad-hoc loaded modules a widgets.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

at the time

@@ -0,0 +1,59 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

chybí contributor, keywords a "es" entry point

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

import invariant from 'invariant';
import injectionMiddleware from './injectionMiddleware';

const createReducers = asyncReducers =>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tome, asi se na mě naštveš, ale me ten zápis ze snipetu příjde méně přehledný :(

const cancelSaga = key => {
if (!isNil(store.injectedSagas[key])) {
store.injectedSagas[key].task.cancel();
store.injectedSagas[key] = null;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@tommmyy tommmyy merged commit c9edc3d into master Feb 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants