-
Notifications
You must be signed in to change notification settings - Fork 7
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
#165273536: Pagination support for articles #41
base: develop
Are you sure you want to change the base?
Conversation
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.
Some files could not be reviewed due to errors:
The configured stylelint version is not supported.
The configured stylelint version is not supported. See a list of supported versions on our docs page.
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.
Some files could not be reviewed due to errors:
The configured stylelint version is not supported.
The configured stylelint version is not supported. See a list of supported versions on our docs page.
@@ -0,0 +1,65 @@ | |||
// react | |||
import React from 'react'; |
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.
Parsing error: 'import' and 'export' may appear only with 'sourceType: module'
@@ -0,0 +1,49 @@ | |||
import React from 'react'; |
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.
Parsing error: 'import' and 'export' may appear only with 'sourceType: module'
03d44f1
to
2063e73
Compare
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.
Some files could not be reviewed due to errors:
The configured stylelint version is not supported.
The configured stylelint version is not supported. See a list of supported versions on our docs page.
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.
Some files could not be reviewed due to errors:
The configured stylelint version is not supported.
The configured stylelint version is not supported. See a list of supported versions on our docs page.
- add article pagination - Revise article like count prop validation from array to object since it has been modified in the back end [starts #165273536]
2063e73
to
3c51443
Compare
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.
Some files could not be reviewed due to errors:
The configured stylelint version is not supported.
The configured stylelint version is not supported. See a list of supported versions on our docs page.
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.
Some files could not be reviewed due to errors:
The configured stylelint version is not supported.
The configured stylelint version is not supported. See a list of supported versions on our docs page.
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.
Some files could not be reviewed due to errors:
The configured stylelint version is not supported.
The configured stylelint version is not supported. See a list of supported versions on our docs page.
@@ -115,15 +156,25 @@ LandingPage.defaultProps = { | |||
* @return {articles} - An aray of articles | |||
* | |||
*/ | |||
const mapStateToProps = state => state.articles; | |||
// const mapStateToProps = state => state.articles; |
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 line.
What does this PR do?
Description of Task to be completed?
How should this be manually tested?
Any background context you want to provide?
since it has been modified in the back end.
What are the relevant pivotal tracker stories?
Screenshots (if appropriate)