Skip to content

Commit 9d5a89d

Browse files
eemaanamireemaanamir
authored
feat: removed flag for new profile UI and depreacted old code (#1233)
1 parent 762e290 commit 9d5a89d

File tree

120 files changed

+3556
-19968
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

120 files changed

+3556
-19968
lines changed

‎.env

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,5 +31,4 @@ SEARCH_CATALOG_URL=''
3131
ENABLE_SKILLS_BUILDER_PROFILE=''
3232
# Fallback in local style files
3333
PARAGON_THEME_URLS={}
34-
ENABLE_NEW_PROFILE_VIEW=''
3534
DISABLE_VISIBILITY_EDITING=''

‎.env.development

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,5 +32,4 @@ SEARCH_CATALOG_URL='http://localhost:18000/courses'
3232
ENABLE_SKILLS_BUILDER_PROFILE=''
3333
# Fallback in local style files
3434
PARAGON_THEME_URLS={}
35-
ENABLE_NEW_PROFILE_VIEW=''
3635
DISABLE_VISIBILITY_EDITING=''

‎.env.test

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,5 +26,4 @@ COLLECT_YEAR_OF_BIRTH=true
2626
APP_ID=''
2727
MFE_CONFIG_API_URL=''
2828
PARAGON_THEME_URLS={}
29-
ENABLE_NEW_PROFILE_VIEW=''
3029
DISABLE_VISIBILITY_EDITING=''

‎src/data/reducers.js

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,9 @@
11
import { combineReducers } from 'redux';
22

3-
import { getConfig } from '@edx/frontend-platform';
4-
53
import { reducer as profilePageReducer } from '../profile';
6-
import { reducer as newProfilePageReducer } from '../profile-v2';
7-
8-
const isNewProfileEnabled = getConfig().ENABLE_NEW_PROFILE_VIEW;
94

105
const createRootReducer = () => combineReducers({
11-
profilePage: isNewProfileEnabled ? newProfilePageReducer : profilePageReducer,
6+
profilePage: profilePageReducer,
127
});
138

149
export default createRootReducer;

‎src/data/sagas.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,8 @@
11
import { all } from 'redux-saga/effects';
2-
import { getConfig } from '@edx/frontend-platform';
32
import { saga as profileSaga } from '../profile';
4-
import { saga as newProfileSaga } from '../profile-v2';
5-
6-
const isNewProfileEnabled = getConfig().ENABLE_NEW_PROFILE_VIEW;
73

84
export default function* rootSaga() {
95
yield all([
10-
isNewProfileEnabled ? newProfileSaga() : profileSaga(),
6+
profileSaga(),
117
]);
128
}

‎src/index-v2.scss

Lines changed: 0 additions & 6 deletions
This file was deleted.

‎src/index.jsx

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import {
77
initialize,
88
mergeConfig,
99
subscribe,
10-
getConfig,
1110
} from '@edx/frontend-platform';
1211
import {
1312
AppProvider,
@@ -28,20 +27,16 @@ import Head from './head/Head';
2827

2928
import AppRoutes from './routes/AppRoutes';
3029

30+
import './index.scss';
31+
3132
const rootNode = createRoot(document.getElementById('root'));
3233
subscribe(APP_READY, async () => {
33-
const isNewProfileEnabled = getConfig().ENABLE_NEW_PROFILE_VIEW === 'true';
34-
if (isNewProfileEnabled) {
35-
await import('./index-v2.scss');
36-
} else {
37-
await import('./index.scss');
38-
}
3934
rootNode.render(
4035
<AppProvider store={configureStore()}>
4136
<Head />
4237
<Header />
4338
<main id="main">
44-
<AppRoutes isNewProfileEnabled={isNewProfileEnabled} />
39+
<AppRoutes />
4540
</main>
4641
<FooterSlot />
4742
</AppProvider>,
@@ -61,7 +56,6 @@ initialize({
6156
mergeConfig({
6257
COLLECT_YEAR_OF_BIRTH: process.env.COLLECT_YEAR_OF_BIRTH,
6358
ENABLE_SKILLS_BUILDER_PROFILE: process.env.ENABLE_SKILLS_BUILDER_PROFILE,
64-
ENABLE_NEW_PROFILE_VIEW: process.env.ENABLE_NEW_PROFILE_VIEW || null,
6559
}, 'App loadConfig override handler');
6660
},
6761
},

‎src/index.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@
33
@import "~@edx/frontend-component-header/dist/index";
44
@import "~@edx/frontend-component-footer/dist/footer";
55

6-
@import './profile/index';
6+
@import 'profile/index';

‎src/profile-v2/DateJoined.jsx

Lines changed: 0 additions & 29 deletions
This file was deleted.

‎src/profile-v2/NotFoundPage.jsx

Lines changed: 0 additions & 16 deletions
This file was deleted.

0 commit comments

Comments
 (0)