-
-
Notifications
You must be signed in to change notification settings - Fork 357
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement Ducks: Modular Redux #421
Conversation
spec/fixtures/site/_config.yml
Outdated
@@ -1,7 +1,7 @@ | |||
title: Your awesome title | |||
|
|||
# Load the admin interface. | |||
plugins: | |||
gems: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CAUTION: This reverts a recent change..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right. But it gives me Deprecation: The 'plugins' configuration option has been renamed to 'plugins_dir'. Please update your config file accordingly.
error. How do I fix this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's probably because your Gemfile.lock
is still using an older version of Jekyll. Running bundle update
will remove the warning.
On an unrelated side-note, once PR #322
gets merged, every other developer locked to an older version will thenceforth not see the warning..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that was it. bundle update jekyll
solved the issue. Thank you 👍
With our current redux setup, if we want to add a new reducer, we have to add
actionTypes
,actions
andreducer
into separate files. This is a really tedious work. It makes more sense for these pieces to be bundled together in an isolated module.With this PR, I migrated current structure to ducks model which is a modular way of creating reducers.