Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Remove dispatchProps from redux #9309

Closed
NejcZdovc opened this issue Jun 7, 2017 · 0 comments
Closed

Remove dispatchProps from redux #9309

NejcZdovc opened this issue Jun 7, 2017 · 0 comments
Assignees
@NejcZdovc
Labels
QA/no-qa-needed refactoring/redux-component release-notes/exclude
Milestone
0.18.x Release

Comments

@NejcZdovc
Copy link
Contributor

Describe the issue you encountered:
Remove dispatchProps from mergeProps function. It's not used.
@NejcZdovc NejcZdovc added this to the 0.18.x milestone Jun 7, 2017
@NejcZdovc NejcZdovc self-assigned this Jun 7, 2017
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jun 7, 2017
@NejcZdovc
Removes dispatchProps from redux
9d6e61a 
Resolves brave#9309

Auditors: @bridiver

Test Plan:
- test should pass
@NejcZdovc NejcZdovc mentioned this issue Jun 7, 2017
Merged
8 tasks
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jun 7, 2017
@NejcZdovc
Removes dispatchProps from redux
7e20e94 
Resolves brave#9309

Auditors: @bridiver

Test Plan:
- test should pass
NejcZdovc added a commit to NejcZdovc/browser-laptop that referenced this issue Jun 7, 2017
@NejcZdovc
Removes dispatchProps from redux
62d6587 
Resolves brave#9309

Auditors: @bridiver

Test Plan:
- test should pass
@NejcZdovc NejcZdovc mentioned this issue Jun 7, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@NejcZdovc NejcZdovc

Labels
QA/no-qa-needed refactoring/redux-component release-notes/exclude
Projects
None yet
Milestone
0.18.x Release
Development

No branches or pull requests
1 participant
@NejcZdovc