brandingbrand / flagship / 3808

62%

push

travis-ci

chore: merge develop into master (#838)

* chore(fscomponents): const name to clearicon from cancelicon

* chore: refactor ccard validation to remove moment

This refactors the validation in the CreditCardForm component to eliminate the need for the moment library. Because we were using moment to simply grab the month and date off a MM/YY formatted string it was easy to change. This helps us achieve our goal of removing dependencies to moment because it's a huge library.

* fix(fscomponents): flatten validation for cc number to onblur

* chore: tell greenkeeper to ignore react-native updates

This updates the Greenkeeper configuration to always ignore react-native updates, as we should never be merging in react-native updates without developer oversight. Syntax per the Greenkeeper docs: https://greenkeeper.io/docs.html#monorepo

* feat: android init config

add various config based values that inject into Android template
this allows for modifying the Android project during the init command

* fix: update react-native-htmlview to version 0.14.0

* feat(fsweb): Enable Progressive Web App

Activates features required to build as a progressive web app.

Also switches to Workbox, addressing issue #620

* fix(flagship): Remove deprecated lifecycle functions

Removing componentWillReceiveProps and componentWillMount.

* fix(fsweb): Match routers path-to-regexp

The react-router uses path-to-regexp@^1.7.0, which generates paths differently from 2.4.0, which fsweb was using without specifying the version. Without this change, the router wasn't working, so you couldn't visit any non-root pages.

* fix(pirateship): Use componentDidUpdate in PSProductDetail

getDerivedStateFromProps wasn't properly propogating the commerceData property, so visiting a PDP without refreshing the page caused it to stay in a loading state forever. Switching to componentDidUpdate resolves the issue.

Resolves https://github.com/brandingbrand/flags... (continued)

550 of 1639 branches covered (33.56%)

Branch coverage included in aggregate %.

929 of 2275 relevant lines covered (40.84%)

5.2 hits per line