Fix walkthough state caching (#527)

* fix state caching between walkthroughs

currently, if a user starts a walkthrough and then, without
performing a refresh of their page, navigates to another walkthrough,
it causes an issue with the manifest of the previous walkthrough
being used to provision services in the new walkthrough.

this results in services not being provisioned when they are
required in walkthroughs.

this change ensures that navigating back to the main page will
ensure the manifest cache for the previous walkthrough is cleared.

verification:
- have an existing rhmi installation from master
- update the web app image to point to quay.io/aidenkeating/tutorial-web-app:fix-caching-2
- open walkthrough 2 (3scale) to the first page, no services should be provisioned
- without refreshing, navigate to walkthrough 1, ensure that fuse and amq are provisioned

* bump version to 2.19.4

Author: aidenkeating

package.json

@@ -1,6 +1,6 @@
 {
   "name": "integreatly-web-app",
-  "version": "2.19.3",
+  "version": "2.19.4",
   "private": true,
   "proxy": "http://localhost:5001/",
   "dependencies": {

src/pages/landing/landingPage.js

@@ -18,8 +18,9 @@ import { DISPLAY_SERVICES } from '../../services/middlewareServices';
 
 class LandingPage extends React.Component {
   componentDidMount() {
-    const { getProgress, getCustomWalkthroughs } = this.props;
+    const { getProgress, getCustomWalkthroughs, resetCurrentWalkthrough } = this.props;
     getCustomWalkthroughs();
+    resetCurrentWalkthrough();
     getProgress();
   }
 
@@ -82,6 +83,7 @@ class LandingPage extends React.Component {
 LandingPage.propTypes = {
   getProgress: PropTypes.func,
   getCustomWalkthroughs: PropTypes.func,
+  resetCurrentWalkthrough: PropTypes.func,
   middlewareServices: PropTypes.object,
   walkthroughServices: PropTypes.object,
   user: PropTypes.object,
@@ -95,6 +97,7 @@ LandingPage.propTypes = {
 LandingPage.defaultProps = {
   getProgress: noop,
   getCustomWalkthroughs: noop,
+  resetCurrentWalkthrough: noop,
   middlewareServices: {
     customServices: {},
     data: {}
@@ -111,6 +114,7 @@ LandingPage.defaultProps = {
 const mapDispatchToProps = dispatch => ({
   getWalkthroughs: language => dispatch(reduxActions.walkthroughActions.getWalkthroughs(language)),
   getCustomWalkthroughs: () => dispatch(reduxActions.walkthroughActions.getCustomWalkthroughs()),
+  resetCurrentWalkthrough: () => dispatch(reduxActions.threadActions.resetCustomThread()),
   getProgress: () => dispatch(reduxActions.userActions.getProgress()),
   launchAMQOnline: (username, namespace) => provisionAMQOnline(dispatch, username, namespace),
   launchAMQOnlineV4: (username, namespace) => provisionAMQOnlineV4(dispatch, username, namespace)

src/redux/actions/threadActions.js

@@ -7,6 +7,11 @@ const getThread = (language, id) => ({
   payload: threadServices.getThread(language, id)
 });
 
+const resetCustomThread = () => ({
+  type: threadTypes.CLEAR_THREAD,
+  payload: Promise.resolve({})
+});
+
 const getCustomThread = id => ({
   type: threadTypes.GET_THREAD,
   payload: threadServices.getCustomThread(id)
@@ -44,5 +49,6 @@ export {
   updateThreadProgress,
   initCustomThreadSuccess,
   initCustomThreadFailure,
-  initCustomThreadPending
+  initCustomThreadPending,
+  resetCustomThread
 };

src/redux/constants/threadConstants.js

@@ -2,5 +2,6 @@ const GET_THREAD = 'GET_THREAD';
 const INIT_THREAD = 'INIT_THREAD';
 const UPDATE_THREAD_PROGRESS = 'UPDATE_THREAD_PROGRESS';
 const UPDATE_CURRENT_THREAD = 'UPDATE_CURRENT_THREAD';
+const CLEAR_THREAD = 'CLEAR_THREAD';
 
-export { GET_THREAD, INIT_THREAD, UPDATE_THREAD_PROGRESS, UPDATE_CURRENT_THREAD };
+export { GET_THREAD, INIT_THREAD, UPDATE_THREAD_PROGRESS, UPDATE_CURRENT_THREAD, CLEAR_THREAD };

src/redux/reducers/threadReducers.js

@@ -24,6 +24,9 @@ const initialState = {
 
 const threadReducers = (state = initialState, action) => {
   switch (action.type) {
+    case FULFILLED_ACTION(threadTypes.CLEAR_THREAD):
+      return Object.assign({}, initialState, { threadProgress: state.threadProgress });
+
     // Error/Rejected
     case REJECTED_ACTION(threadTypes.GET_THREAD):
       return setStateProp(