newsuk / times-components / 6506

89%

push

travis-ci

feat: REPLAT-1403 - Article Summary Refactor (#586)

* feat: initial commit of refactoring article summary to a factory driven API

* chore: adds remaining fixtures to article summary

* chore: make it pretty

* chore: making linter happier

* chore: fixes errors in fixtures

* chore: update snapshots in article summary

* chore: adds multiple exports to article-summary

* feat: sync up author-profile-item to use new article-summary api

* feat: sync up author-profile-item to use new article-summary-web api

* chore: updates fixtures of card to use new article summary api

* chore: make it pretty

* chore: updates author-profile to use new article-summary api

* chore: Code Review, linting and formatting

* test: Update of web tests

* chore: Remove unused styles

* chore: rename textAst to summaryText in ArticleSummary

* chore: refactors renderTrees

* chore: refactors renderTrees in author-profile

* chore: moves ternery in author-profile outside jsx

* chore: making things prettier

* Feat: REPLAT-1388 - Date Publication refactor (#587)

* chore: Update DatePublication to be more flexible

* chore: Removing unused props from DatePublication

* test: Updating Date Publication to use jest-configurator

* chore: Linting and formatting

* chore: removes publication from slice

* chore: update snapshot in slice

* chore: refactors article-summary-headline

* chore: updates snapshots of slice

* fix: fixing native styles of article-summary-headline

* fix: Updating the visual styling required for making everything look as it previously did cross platform

* fix: Added turnery operators to avoid showing empty containers in the snapshots

* chore: Removing snapshots to aid merge

* Test: Update the Article Summary package to enable Jest Configurator (#588)

* feat: add proptype documentation in storybook (#572)

* test: Update the Article Summary package to enable Jest Configurator

* refactor: add flow to tracking context file (#549)

* chore: Publish 9c91141

 - @times-components/article@0.31.4
 - @times-components/author-head@0.17.12
 - @times-components/author-profile@0.41.12
 - @times-components/pagination@0.12.39
 - @times-components/tracking@0.6.11

* chore: Updating test conventions on Article Summary

* feat: add provider testing package (#543)

* chore: Publish d563b83

 - @times-components/provider-test-tools@0.1.0

* chore: updated snapshots

* feat: initial commit of refactoring article summary to a factory driven API

* chore: adds remaining fixtures to article summary

* chore: make it pretty

* chore: making linter happier

* chore: fixes errors in fixtures

* chore: update snapshots in article summary

* chore: adds multiple exports to article-summary

* feat: sync up author-profile-item to use new article-summary api

* feat: sync up author-profile-item to use new article-summary-web api

* chore: updates fixtures of card to use new article summary api

* chore: make it pretty

* chore: updates author-profile to use new article-summary api

* chore: Code Review, linting and formatting

* test: Update of web tests

* chore: Remove unused styles

* chore: rename textAst to summaryText in ArticleSummary

* chore: linting and formatting

* chore: refactors renderTrees

* chore: moves ternery in author-profile outside jsx

* chore: making things prettier

* chore: removes publication from slice

* chore: update snapshot in slice

* chore: refactors article-summary-headline

* fix: fixing native styles of article-summary-headline

* fix: Updating the visual styling required for making everything look as it previously did cross platform

* fix: Added turnery operators to avoid showing empty containers in the snapshots

* chore: Removing snapshots to aid merge

* Test: Update the Article Summary package to enable Jest Configurator (#588)

* feat: add proptype documentation in storybook (#572)

* test: Update the Article Summary package to enable Jest Configurator

* refactor: add flow to tracking context file (#549)

* chore: Publish 9c91141

 - @times-components/article@0.31.4
 - @times-components/author-head@0.17.12
 - @times-components/author-profile@0.41.12
 - @times-components/pagination@0.12.39
 - @times-components/tracking@0.6.11

* chore: Updating test conventions on Article Summary

* feat: add provider testing package (#543)

* chore: Publish d563b83

 - @times-components/provider-test-tools@0.1.0

* chore: linting and formatting

* chore: cherrypick after rebase

* chore: trying to clean up the PR...

* chore: fixing the article-image changelog

* chore: step by step fixing the messy rebase

* chore: last of the rebase cleanup

* chore: minor fix and test updates to date-publication

* chore: More minor fixes to packages to keep up with master

* feat: Allow for more flexible definition of publication in DatePublication

* chore: Linting and formatting

* chore: Minor fixes

* chore: Styling fixes across Slice and Article summary

* chore: Updating tests across the board

* chore: Yarn lock and linting

* chore: Apply formatting across the packages for passing the linter

* test: Increase coverage on article-summary

* test: Update slice test code and snapshots

* chore: linting and formatting cross packages

* chore: Update dependencies missed by rebase

* fix: fix the author profile tests

* fix: fix the author profile tests

* chore: switching tests to new api

* chore: switching author-profile to new article-summary api

* chore: Refactor Slice to use the new Article Summary api

* chore: update tests, linting and formatting

* chore: update Card article summary props

* chore: update formatting

* chore: implement the initial code review feedback

* test: improve the test coverage on the article summary package

* chore: apply lormatting

* chore: fix the linting problems

* chore: post rebase fixes

* chore: Updating Slice snapshot tests

590 of 646 branches covered (91.33%)

Branch coverage included in aggregate %.

1896 of 1931 relevant lines covered (98.19%)

14.31 hits per line