Skip to content

Conversation

@miiu96
Copy link
Owner

@miiu96 miiu96 commented Oct 21, 2019

first pr

{this.Arr1.map((item, index) => (<this.renderListItem key={item.id} item={item} />))}
<button onClick={() => console.log(this.props.stateprop1)}> GetState</button>
<button onClick={() => this.props.action1()}> Dispatch Action 1</button>
<button onClick={() => this.props.action1()}> Dispatch Action 2</button>

Choose a reason for hiding this comment

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

you can define a var instead of using the same string


function mapDispatchToProps(dispatch) {
return {
action1: () => dispatch(ACTIONS.SUCCESS),

Choose a reason for hiding this comment

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

maybe a better naming for the functions

import React from 'react';
import ReactDOM from 'react-dom';
import App from './App'

Choose a reason for hiding this comment

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

remove empty line

Choose a reason for hiding this comment

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

not done yet

@@ -0,0 +1,3 @@
export const SUCCESS = "SUCCESS"

export const FAILURE = "FAILURE" No newline at end of file

Choose a reason for hiding this comment

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

add empty line at the end of the file

Choose a reason for hiding this comment

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

not done yet

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.

3 participants