-
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
#165273545: Viewing read time for Articles or an Article #36
base: develop
Are you sure you want to change the base?
Conversation
this commit adds a feature for creating an article files contained in this commit; - Articles folder that contains the js and scss for articles pages [Maintains #165273532]
this commit adds a feature for getting an article included in this commit; - UI design for an article page [Maintains #165273532]
this commit adds a feature for creating an article files contained in this commit; - Articles folder that contains the js and scss for articles pages [Maintains #165273532]
this commit adds a feature for viewing all articles [Maintains #165273532]
this commit does the following; - adds feature creating an article - adds feature for getting all articles
this commit adds the following features to the application; - a feature for creating an article - a feature for getting all articles - a feature for getting an article - all features added have UI pages they are linked to [Maintains #165273532]
this commit adds a feature for seeing the read time of an article [Maintains #165273545]
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.
export const axiosConfig = { | ||
headers: { | ||
'Content-Type': 'application/json', | ||
Authorization: `Bearer ${sessionStorage.getItem('ahToken')}`, |
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.
'sessionStorage' is not defined no-undef
export const baseURL = 'https://ah-backend-dojo-dev.herokuapp.com/api'; | ||
|
||
export const defaultImgURL = 'https://ah-frontend-dojo.herokuapp.com/src/asset'; | ||
export const author = sessionStorage.getItem('username'); |
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.
'sessionStorage' is not defined no-undef
expect(newState.articles.length).toEqual(0); | ||
}); | ||
|
||
it('Should return the passed state', () => { | ||
const newState = articles(initialSate, generatePayload(GET_ARTICLES)); | ||
const newState = articlesReducer(initialSate, generatePayload(GET_ARTICLES)); | ||
expect(newState.articles.length).toEqual(0); |
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.
'expect' is not defined no-undef
}); | ||
|
||
it('should edit an article', () => { | ||
expect( |
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.
'expect' is not defined no-undef
}); | ||
|
||
it('should edit an article', () => { | ||
expect( |
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.
'expect' is not defined no-undef
import { getAllArticles } from 'store/actions/articleActions'; | ||
import { GET_ARTICLES, GET_ARTICLES_START } from '../articleTypes'; | ||
import { getAllArticles, newArticle } from 'store/actions/articleActions'; | ||
import data from 'store/actions/__mocks__'; |
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.
'data' is defined but never used no-unused-vars
import thunk from 'redux-thunk'; | ||
import { getAllArticles } from 'store/actions/articleActions'; | ||
import { GET_ARTICLES, GET_ARTICLES_START } from '../articleTypes'; | ||
import { getAllArticles, newArticle } from 'store/actions/articleActions'; |
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.
'newArticle' is defined but never used no-unused-vars
getArticle(slug); | ||
} | ||
|
||
fetchArticle = (e) => { |
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: Unexpected token =
export default class Search extends Component { | ||
render() { | ||
return ( | ||
<div className="searchPanel"> |
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: Unexpected token <
import React from 'react'; | ||
import './Loader.scss'; | ||
|
||
const ProgressLoader = () => <div className="progress__loader" />; |
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: Unexpected token <
What does this PR do?
Description of Task to be completed?
How should this be manually tested?
npm install
npx jest --coverage -u
npm run dev
Any background context you want to provide?
What are the relevant pivotal tracker stories?
-165273545
Screenshots (if appropriate)
Read time display for all articles
Read time display for an article