-
Notifications
You must be signed in to change notification settings - Fork 4
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
#165412889 & #165412878 Add article rating and article read time #23
Conversation
@@ -80,12 +87,13 @@ export class PreviewArticle extends Component { | |||
}; | |||
|
|||
render() { | |||
const { message } = this.props; | |||
const { message, loading } = this.props; |
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.
'loading' is missing in props validation react/prop-types
import configureMockStore from 'redux-mock-store'; | ||
import thunk from 'react-thunk'; | ||
import { Provider } from 'react-redux'; | ||
// import article from '../../../__mocks__/article'; |
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.
remove the comment
const result = createRate(rating)(dispatch); | ||
expect(result).toHaveProperty('type'); | ||
expect(result).toHaveProperty('payload'); | ||
// expect(result.payload).toEqual(rating); |
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.
the same here
// import article from '../../__mocks__/article'; | ||
// import message from '../../__mocks__/articleMessage'; | ||
// import errors from '../../__mocks__/errors'; | ||
|
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.
Please make sure that all comments are removed
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.
remove comments
03870b9
to
b837925
Compare
faTrash, | ||
faPen, | ||
faTimesCircle, | ||
faCircle, |
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.
'faCircle' is defined but never used no-unused-vars
import { | ||
faTrash, | ||
faPen, | ||
faTimesCircle, |
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.
'faTimesCircle' is defined but never used no-unused-vars
import PropTypes from 'prop-types'; | ||
import { FontAwesomeIcon } from '@fortawesome/react-fontawesome'; | ||
import { | ||
faTrash, |
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.
'faTrash' is defined but never used no-unused-vars
9a9a18b
to
4148a9e
Compare
4148a9e
to
b7a3540
Compare
b7a3540
to
8eddf16
Compare
#165412889 & #165412878 Add article rating and article read time
#165412889 & #165412878 Add article rating and article read time
#165412889 & #165412878 Add article rating and article read time
What does this PR do?
Description of Task to be completed?
How should this be manually tested?
After cloning the repo,
run
the commandnpm run dev
, and navigate to an article of your choice from the home pageAny background context you want to provide?
N/A
What are the relevant pivotal tracker stories?
#165412889
#165412878
Screenshots (if appropriate)
Questions: