Merge pull request #1951 from appirio-tech/hotfix/v5-notifications-endpoint-change

[Production Deployment] hotfix: v5 deployment - changes in tc-notifications endpoints.

Author: sachin-maheshwari

config/constants/dev.js

@@ -44,5 +44,5 @@ module.exports = {
   IBM_COGNITIVE_PROGRAM_ID  : 3449,
   HEAP_ANALYTICS_APP_ID     : '4153837120',
 
-  TC_NOTIFICATION_URL: 'https://api.topcoder-dev.com'
+  TC_NOTIFICATION_URL: 'https://api.topcoder-dev.com/v5/notifications'
 }

config/constants/master.js

@@ -43,5 +43,5 @@ module.exports = {
   IBM_COGNITIVE_PROGRAM_ID  : 3449,
   HEAP_ANALYTICS_APP_ID     : '638908330',
 
-  TC_NOTIFICATION_URL: 'https://api.topcoder.com'
+  TC_NOTIFICATION_URL: 'https://api.topcoder.com/v5/notifications'
 }

config/constants/qa.js

@@ -42,5 +42,5 @@ module.exports = {
   IBM_COGNITIVE_PROGRAM_ID  : 3449,
   HEAP_ANALYTICS_APP_ID     : '4153837120',
 
-  TC_NOTIFICATION_URL: 'http://api.topcoder-dev.com/'
+  TC_NOTIFICATION_URL: 'https://api.topcoder-dev.com/v5/notifications'
 }

src/config/constants.js

@@ -297,7 +297,7 @@ export const ACCOUNTS_APP_REGISTER_URL = process.env.ACCOUNTS_APP_REGISTER_URL |
 export const TC_API_URL = `https://api.${DOMAIN}`
 export const DIRECT_PROJECT_URL = `https://www.${DOMAIN}/direct/projectOverview?formData.projectId=`
 export const SALESFORCE_PROJECT_LEAD_LINK = process.env.SALESFORCE_PROJECT_LEAD_LINK
-export const TC_NOTIFICATION_URL = process.env.TC_NOTIFICATION_URL || TC_API_URL
+export const TC_NOTIFICATION_URL = process.env.TC_NOTIFICATION_URL || `${TC_API_URL}/v5/notifications`
 
 export const PROJECT_NAME_MAX_LENGTH = 255
 export const PROJECT_REF_CODE_MAX_LENGTH = 32

src/routes/notifications/services/notifications.js

@@ -10,19 +10,19 @@ import { prepareNotifications } from '../helpers/notifications'
 // the id can be either: null/undefined (mark all); notification id; or '-' separated ids, e.g. '123-456-789'
 const markNotificationsRead = (id) => {
   if (id) {
-    return axios.put(`${TC_NOTIFICATION_URL}/notifications/${id}/read`)
+    return axios.put(`${TC_NOTIFICATION_URL}/${id}/read`)
   } else {
-    return axios.put(`${TC_NOTIFICATION_URL}/notifications/read`)
+    return axios.put(`${TC_NOTIFICATION_URL}/read`)
   }
 }
 
 // the id can be either: notification id; or '-' separated ids, e.g. '123-456-789'
 const markNotificationsSeen = (id) => {
-  return axios.put(`${TC_NOTIFICATION_URL}/notifications/${id}/seen`)
+  return axios.put(`${TC_NOTIFICATION_URL}/${id}/seen`)
 }
 
 const getNotifications = () => {
-  return axios.get(`${TC_NOTIFICATION_URL}/notifications?read=false&limit=${NOTIFICATIONS_LIMIT}`)
+  return axios.get(`${TC_NOTIFICATION_URL}/list?read=false&limit=${NOTIFICATIONS_LIMIT}`)
     .then(resp => prepareNotifications(resp.data.items))
 }
 

src/routes/settings/services/settings.js

@@ -62,7 +62,7 @@ const changePassword = (password) => {
 }
 
 const getNotificationSettings = () => {
-  return axios.get(`${TC_NOTIFICATION_URL}/notificationsettings`)
+  return axios.get(`${TC_NOTIFICATION_URL}/settings`)
     .then(resp => resp.data)
 }
 
@@ -101,7 +101,7 @@ const saveNotificationSettings = (data) => {
     body.push({ topic, deliveryMethod: 'email', value: data[topic] && data[topic].email === 'yes' ? 'yes' : 'no' })
     body.push({ topic, deliveryMethod: 'web', value: data[topic] && data[topic].web === 'yes' ? 'yes' : 'no' })
   })
-  return axios.put(`${TC_NOTIFICATION_URL}/notificationsettings`, body)
+  return axios.put(`${TC_NOTIFICATION_URL}/settings`, body)
 }
 
 export default {