Skip to content
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

Switches to object config argument syntax over mutli args #12

Merged
merged 6 commits into from
Feb 3, 2016

Conversation

mjrussell
Copy link
Owner

Resolves #8

Allows for far more variety of config options and lets the user set the fields they really want. Previously if the user wanted allowRedirectBack to be false, they had to specify predicate even if they wanted the default behavior.

const UserIsAuthenticated = UserAuthWrapper(state => state.user)('/login', 'UserIsAuthenticated')

becomes

const UserIsAuthenticated = UserAuthWrapper({
  authSelector: state => state.user,
  wrapperDisplayName: 'UserIsAuthenticated'
})

Preserves a backwards (deprecated) compatible version.

Updated tests, README, and examples.

mjrussell added a commit that referenced this pull request Feb 3, 2016
Switches to object config argument syntax over mutli args
@mjrussell mjrussell merged commit 6be9554 into master Feb 3, 2016
@mjrussell mjrussell deleted the object-args branch February 3, 2016 00:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant