Merge pull request #790 from maxceem/issue-774

#774 - fix: update challenge in the list after editing

Author: vikasrohit

src/reducers/challenges.js

@@ -85,8 +85,20 @@ export default function (state = initialState, action) {
       return { ...state, isLoading: false, attachments: [], challenge: null, failedToLoad: true }
     case LOAD_CHALLENGE_DETAILS_SUCCESS:
     case UPDATE_CHALLENGE_DETAILS_SUCCESS:
+      let updatedChallenges = state.challenges
+      // when updating challenge details, make the same update in the challenge list
+      if (action.type === UPDATE_CHALLENGE_DETAILS_SUCCESS) {
+        const updatedChallengeIndex = _.findIndex(state.challenges, { id: action.challengeDetails.id })
+        updatedChallenges = [
+          ...state.challenges.slice(0, updatedChallengeIndex),
+          action.challengeDetails,
+          ...state.challenges.slice(updatedChallengeIndex + 1)
+        ]
+      }
+
       return {
         ...state,
+        challenges: updatedChallenges,
         challengeDetails: action.challengeDetails,
         isLoading: false,
         attachments: _.has(action.challengeDetails, 'attachments') ? action.challengeDetails.attachments : [],

src/services/challenges.js

@@ -138,7 +138,11 @@ export function fetchChallenges (filters, params) {
     ...filters,
     ...params
   }
-  return axiosInstance.get(`${CHALLENGE_API_URL}?${qs.stringify(query, { encode: false })}`)
+  return axiosInstance.get(`${CHALLENGE_API_URL}?${qs.stringify(query, { encode: false })}`).then(response => {
+    // normalize challenge data in the list of challenges for consistency with data of a single challenge details page
+    response.data = response.data.map(normalizeChallengeDataFromAPI)
+    return response
+  })
 }
 
 /**