fix: Remove fetchStripeCustomerPortalURL related logic

edx · Jul 30, 2024 · a8da25d · a8da25d
1 parent 6fe4f62
commit a8da25d
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 51 deletions.
diff --git a/src/store/__mocks__/mockEmptyStore.js b/src/store/__mocks__/mockEmptyStore.js
@@ -13,7 +13,6 @@ module.exports = {
     loading: false,
     loadingError: false,
     subscriptions: [],
-    stripeCustomerPortalURL: null,
     stripeError: false,
     stripeLoading: false,
     shouldShowSubscriptionsSection: true,

diff --git a/src/store/__mocks__/subscriptions.mockStore.js b/src/store/__mocks__/subscriptions.mockStore.js
@@ -4,7 +4,6 @@ module.exports = {
     loadingError: false,
     stripeLoading: false,
     stripeError: false,
-    stripeCustomerPortalURL: null,
     shouldShowSubscriptionsSection: true,
     subscriptions: [
       {

diff --git a/src/subscriptions/actions.js b/src/subscriptions/actions.js
@@ -1,9 +1,6 @@
 import { createRoutine } from 'redux-saga-routines';
 
 export const fetchSubscriptions = createRoutine('FETCH_SUBSCRIPTIONS');
-export const fetchStripeCustomerPortalURL = createRoutine(
-  'FETCH_STRIPE_CUSTOMER_PORTAL_URL',
-);
 
 export const clearStripeError = () => ({
   type: 'CLEAR_STRIPE_ERROR',

diff --git a/src/subscriptions/reducer.js b/src/subscriptions/reducer.js
@@ -1,10 +1,9 @@
-import { fetchStripeCustomerPortalURL, fetchSubscriptions } from './actions';
+import { fetchSubscriptions } from './actions';
 
 export const initialState = {
   loading: true,
   loadingError: false,
   subscriptions: [],
-  stripeCustomerPortalURL: null,
   stripeError: false,
   stripeLoading: false,
   shouldShowSubscriptionsSection: true,
@@ -33,27 +32,6 @@ const subscriptionsReducer = (state = initialState, action = {}) => {
         ...state,
         loading: false,
       };
-    case fetchStripeCustomerPortalURL.TRIGGER:
-      return {
-        ...state,
-        stripeLoading: true,
-        stripeCustomerPortalURL: null,
-      };
-    case fetchStripeCustomerPortalURL.SUCCESS:
-      return {
-        ...state,
-        stripeCustomerPortalURL: action.payload,
-      };
-    case fetchStripeCustomerPortalURL.FAILURE:
-      return {
-        ...state,
-        stripeError: true,
-      };
-    case fetchStripeCustomerPortalURL.FULFILL:
-      return {
-        ...state,
-        stripeLoading: false,
-      };
     case 'CLEAR_STRIPE_ERROR':
       return {
         ...state,

diff --git a/src/subscriptions/saga.js b/src/subscriptions/saga.js
@@ -1,9 +1,7 @@
 import { put, takeLatest, call } from 'redux-saga/effects';
 
-import { createFetchHandler } from '../utils';
-
-import { fetchSubscriptions, fetchStripeCustomerPortalURL, hideSubscriptionSection } from './actions';
-import { getSubscriptions, getStripeCustomerPortalURL } from './service';
+import { fetchSubscriptions, hideSubscriptionSection } from './actions';
+import { getSubscriptions } from './service';
 
 function* handleFetchSubscriptions() {
   try {
@@ -27,15 +25,6 @@ function* handleFetchSubscriptions() {
   }
 }
 
-const handleFetchStripeCustomerPortalURL = createFetchHandler(
-  fetchStripeCustomerPortalURL,
-  getStripeCustomerPortalURL,
-);
-
 export default function* subscriptionSaga() {
   yield takeLatest(fetchSubscriptions.TRIGGER, handleFetchSubscriptions);
-  yield takeLatest(
-    fetchStripeCustomerPortalURL.TRIGGER,
-    handleFetchStripeCustomerPortalURL,
-  );
 }
diff --git a/src/subscriptions/service.js b/src/subscriptions/service.js
@@ -19,13 +19,3 @@ export async function getSubscriptions() {
         : subscription.subscription_state?.toLowerCase(),
   }));
 }
-
-export async function getStripeCustomerPortalURL() {
-  const httpClient = getAuthenticatedHttpClient();
-
-  const stripePortalURL = createURL('/stripe-portal/');
-
-  const { data } = await httpClient.post(stripePortalURL);
-
-  return data.portal_url;
-}